Closed (fixed)
Project:
Services
Version:
7.x-3.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Reporter:
Created:
30 Jul 2011 at 18:06 UTC
Updated:
9 Aug 2011 at 18:01 UTC
Jump to comment: Most recent file
Updated to 7.x.-3.0-rc4 from rc3.
Keep running update.php, and always getting 2 pending updates Over and Over:
services module
7303 - Update 7303 adds the possibility to configure server settings on a per-endpoint basis. and sets upgrades all new servers to have at least services session enabled.
7399 - Update 7304 removes the title as it is no longer used
I've reverted back to rc3 (went away), reinstalled rc4 (came back with same behaviour). Do I need to go into the database to manually fix this or ...?
Thanks,
Marc
"You spin me right round, baby, right round
like a record baby, right round round round."
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | services-1234484-8.patch | 350 bytes | wedge |
| #4 | services-1234484.patch | 860 bytes | claudiu.cristea |
Comments
Comment #1
marcingy commentedSounds like a duplicate of http://drupal.org/node/1233010 as services just uses core update functionality
Comment #2
wedge commentedI'm seeing the same thing. I think it's a core issue with postgresql. Are you using this marcjohnson?
I'm getting the following message the first time I run update.php:
Failed: PDOException: SQLSTATE[23502]: Not null violation: 7 ERROR: column "server_settings" contains null values: ALTER TABLE {services_endpoint} ALTER server_settings SET NOT NULL; Array ( ) in db_add_field() (line 2781 of /Users/ottob/Sites/drupal7/includes/database/database.inc).The next time it says that the server_settings column already exists, which it does. But only as bytea, no constraints or description.
If I modify includes/database/pgsql/schema.inc about line 371 like this:
I'm a bit short on time right now, but will create a proper issue for core soon.
Comment #3
claudiu.cristeaIt's not duplicate of #1233010: 7.7 update did NOT update!. Reopened.
This is caused because the new field is forced to be NOT NULL.
Comment #4
claudiu.cristeaHere's a patch. Delete the field "server_settings" from "services_endpoint" if exists before rerunning the update script.
Comment #5
wedge commentedBut isn't it reasonable to have the server_settings fields as NOT NULL? In core for instance the parameters fields of the actions table is also bytea NOT NULL. Your solution feels more like a workaround.
Comment #6
wedge commentedReported to core: #1240304: addField issue with not null blobs on PostgreSQL
Comment #7
claudiu.cristeaWhile there are endpoints that doesn't have "server_settings" it seems reasonable for me to allow NULLs for this field. So, not only a workaround...
Comment #8
wedge commentedThis should work if keeping not null is preferred. See Damiens comment in #1240304: addField issue with not null blobs on PostgreSQL.
Comment #9
claudiu.cristeaI don't see the reason of having this field as NOT NULL while there are records where "server_settings" is N/A (the real world "N/A" has to be read as "NULL" in the virtual world). Don't want to insist more on this subject...
Comment #10
sumitk commentedSame with me
Log says:
Comment #11
marcingy commentedAfter reading damz comments I'm happy to rbtc this and bumping to major as this blocks an upgrade path.
Comment #12
kylebrowning commentedIm at a conference until monday, Marcingy if you want to commit, go for it.
Comment #13
kylebrowning commented