I wanted to update the module to your new CVS version
the first error was in line 18:
variable_del('nodewords_global_keywords);
should be
variable_del('nodewords_global_keywords');

and then comes
Fatal error: Call to undefined function: module_exist() in D:\Eigene Dateien\Webserver\drupal_cvs\update-nodewords.php on line 11

please fix it soon
thanks

CommentFileSizeAuthor
#2 update-nodewords.php.patch594 bytesTobias Maier

Comments

Robrecht Jacques’s picture

Robrecht Jacques commented
Assigned: Unassigned » Robrecht Jacques

Fixed in CVS.

Tobias Maier’s picture

Tobias Maier commented
StatusFileSize
new update-nodewords.php.patch594 bytes

have you ever tried your update?
it does not work!

here comes a patch which does it for me!

please commit it and look at the manual for drupal_bootstrap()

thanks again

Tobias Maier’s picture

Tobias Maier commented
Status: Active » Needs review
Tobias Maier’s picture

Tobias Maier commented

why cant the update rename the old table
set up the new one
copy the datas
and delete the old table?

Robrecht Jacques’s picture

Robrecht Jacques commented
Status: Needs review » Fixed

Thanks. Probably fixed in CVS-HEAD now.

why cant the update rename the old table
set up the new one
copy the datas
and delete the old table?

Because there is no database agnostic function for that. The format in PostgreSQL and MySQL is different for creating tables. Furthermore, this doesn't seem to be common practise in Drupal.

Anonymous’s picture

(not verified) commented
Anonymous’s picture

(not verified) commented
Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)