attached is a patch which should successfully update the system table for core module's move to their own directory. untested...

CommentFileSizeAuthor
#4 update_core_modules_move_2.patch1.22 KBJaza
#2 update_core_modules_move_1.patch1.33 KBhunmonk
#1 update_core_modules_move_0.patch1.33 KBhunmonk
update_core_modules_move.patch1.37 KBhunmonk

Comments

hunmonk’s picture

hunmonk commented
Title: modules to own directory breaks core, update neede. » modules to own directory breaks core, update needed.
StatusFileSize
new update_core_modules_move_0.patch1.33 KB

ok, the last one had some problems. this one works, and is tested as working... :)

hunmonk’s picture

hunmonk commented
StatusFileSize
new update_core_modules_move_1.patch1.33 KB

fixing a small coding style mistake...

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Needs review » Needs work

Reviewed the latest patch (not tested). Indeed this is needed. IMHO the modules should not be all on their own line, but on a single line instead. Otherwise this seems to be do what is needed, and will run on start of the update, so it should fix the problem.

Jaza’s picture

Jaza commented
Status: Needs work » Reviewed & tested by the community
StatusFileSize
new update_core_modules_move_2.patch1.22 KB

Tested this patch on an install where the 'system' table in the database has the old file paths, and the file system has the new directory structure (and the old cache is uncleared). Works great.

I agree with Goba - modules should all be on one line. New patch attached that puts all modules on one line. Also updated the 'needed for 4.5 to 4.7 upgrade' comment - now says 'needed for 4.5 to 4.8 upgrade'.

Let's get this in pronto, please.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Anonymous’s picture

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