PostgreSQL surge #4: fix undefined function call
Damien Tournoud - November 24, 2008 - 19:23
| Project: | Drupal |
| Version: | 7.x-dev |
| Component: | postgresql database |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Description
DatabaseSchema_pgsql::addField() calls $this->_createFieldSql() while that function is in fact called $this->createFieldSql().

#1
#2
Seems like there's another one: Call to undefined method DatabaseSchema_pgsql::_processField()
#3
#4
All succeed except 7 fails on schema api
Insert with a default succeeded. Other schema.test 50 SchemaTestCase->testSchema()
Insert into the new table succeeded. Other schema.test 62 SchemaTestCase->testSchema()
Two fields were successfully inserted. Other schema.test 66 SchemaTestCase->testSchema()
Insert with a serial succeeded. Other schema.test 80 SchemaTestCase->testSchema()
Insert with a serial succeeded. Other schema.test 82 SchemaTestCase->testSchema()
The serial is monotone. Other schema.test 84 SchemaTestCase->testSchema()
There were two rows. Other schema.test 87 SchemaTestCase->testSchema()
#5
Ok, patch contains 3 other very small fixes in schema.test which makes all database tests pass,
HOWEVER, someone with more postgres skills should find out why those small fixes make them pass and why a default value of 0 makes it fail ..
#6
Well, no, you will not modify the test case to satisfy the PostgreSQL driver!
#7
I know, I should have marked it CNW also, just wanted to point out with the patch where the failures were :)
#8
This should be better.
#9
Last patch makes all Databases tests pass, so marking rtbc
#10
Yes it works like a charm.
#11
What is the impact of this bug on database creation under PostgreSQL?
Should I delete the Drupal7 database and create a new one from scratch?
#12
Committed to CVS HEAD. Thanks! :)
#13
#14
Agreed with the above. After applying http://drupal.org/node/337794 and this patch I am able to run all simpletests on php5, apache2, postgres 8.1 without this error:
Final Simpletest output:
#15
Automatically closed -- issue fixed for two weeks with no activity.