updates.inc inconsistencies [#44286]

There's been some kind of mixup in the updates.inc file, and a few updates seem to have gone AWOL as a result:

Diff between 4.6.0 and 4.6.2 updates.inc.
Diff between 4.6.2 updates.inc and a much later version.
A related older issue.
Resulting problem.

As a result. all upgraded versions of Drupal are missing the tags column in the vocabulary table.

Cheers
-K

Comment	File	Size	Author
#5	update_19.patch	2.23 KB	Zen
#5
#1	update.php_5.diff	889 bytes	drumm
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

drumm

he/him

NY, US

CreditAttribution: drumm commented 12 January 2006 at 23:39

File	Size
update.php_5.diff	889 bytes

I think this will fix that issue. Please test!

Comment #2

drumm

he/him

NY, US

CreditAttribution: drumm commented 12 January 2006 at 23:40

Status:

Active

» Needs review

Comment #3

Zen CreditAttribution: Zen commented 13 January 2006 at 08:03

Looks good - but I haven't tested this yet. Just wanted to mention that there will very likely be a few adventurous 4.7 beta users out there who might want a fix as well - perhaps with a simple SQL file [for betas 1, 2 and 3].

Cheers
-K

Comment #4

sangamreddi CreditAttribution: sangamreddi commented 14 January 2006 at 20:45

I have tested the dif working good. Thanks a lot. So far no errors. The vocab problem has been solved.

Only error during my update from Drupal 4.6.5 to Drupal latest CVS(Jan 14th) is

Update #157
DELETE FROM {url_alias} WHERE src = 'node/feed' AND dst = 'rss.xml'
Failed: INSERT INTO {url_alias} (src, dst) VALUES ('rss.xml', 'node/feed')

Hope this can easily be add using add alias.

Comment #5

Zen CreditAttribution: Zen commented 14 January 2006 at 21:20

File	Size
update_19.patch	2.23 KB

Tested on 4.6.2 with update_start set as 2005-05-07.
a) the referer field issue with the watchdog still appears if $access_check=TRUE and you're (admin) not logged in.
b) update.php now displays 129 instead of 130 and all updates appear to go through :)
c) However updates 132, 135 and 137 are being performed when they shouldn't be. The attached patch addresses that.

Thanks
-K