DatabaseSchema_pgsql::renameTable() needs to rename the indexes

This patch adds field[id] to index-name so all index names for fields are unique and does not need rename on delete

I still think that we should change index-names inside field_sql_storage_field_storage_delete_field()

Because after renaming table schema is changed and index-names returned by _field_sql_storage_schema() are different from actually stored in DB for all DB-engines

So +1 to commit your patch, I dont know is it a good idea to query pg_indexes... waiting for Josh review

Anyway confirm this fix both tests for fields.

There's a a test testFieldUpdateIndexesWithData which works only for mysql now

Suppose this onу could be extended

field_sql_storage.module

  function getIndexes($table) {
    $indexes = array();
    $result = db_query("SHOW INDEXES FROM {" . $table . "}");
    foreach ($result as $row) {
      $indexes[$row->key_name][$row->seq_in_index] = $row->column_name;
    }
    return $indexes;
  }

Test for renaming indexes #720620: indexExists() for pgsql and sqlite does not prefix tablename

Because PK and UK are indexes this patch works fine.
I think tests from #8 cover index rename

Suppose Damien does not mean to change the api

Extended tests goes in #718254: Fieldname in tests are too long and #720620: indexExists() for pgsql and sqlite does not prefix tablename
Because this is a different trouble

Sequence of patches:
1) This because other issues with tests depends on it
2) #720620: indexExists() for pgsql and sqlite does not prefix tablename - to repair schema (here first part of tests)
3) #718254: Fieldname in tests are too long - Tests for fields storage now could be expanded on posgresql and sqlite

New patch with improved comment

As #720620: indexExists() for pgsql and sqlite does not prefix tablename commited so tests already in.