Closed (fixed)
Project:
View reference
Version:
6.x-2.21
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
12 Jul 2009 at 15:21 UTC
Updated:
22 Sep 2009 at 09:10 UTC
Jump to comment: Most recent file
Comments
Comment #1
g10 commentedseems the following line is missing in the .info file:
core = 6.x
Comment #3
plj commentedThe simplicity of the problem is no reason to leave it unfixed! Patches are required for even simple mistakes!
I upgraded multiple modules on a certain site and then wasted nearly three hours trying to figure out where the problem was after I'd got complaints that information was missing on certain pages; I hadn't built those views, so I didn't understand that View Reference was necessary, and hadn't realised this error, since I hadn't looked into the list of installed modules (modules don't usually disable themselves during updates).
Patch attached; it is against DRUPAL-6--2, i.e. the “dev” tag (I don't know why can't I select that version here). As it is very obvious, I doubt it needs any further testing.
In its current form the 6.x-2.21 cannot be used without manual fixing, so I think this issue should warrant an immediate new release.
Comment #4
danielb commentedI cannot replicate the problem you are having? I don't understand, your patches just adds in something that is already there :/
I just downloaded it - the line is not missing from the file.
Comment #5
danielb commentedThis issue bugs me because I see in my other modules the core line is there twice, once added by me, once by the packaging script. Weird I don't get it. I must have removed it during a CVS update thinking there was redundancy.
But this doesn't explain why the module works for me (and others) when I download and install it, but not for you two.
Comment #6
plj commentedThe reason is most likely that we both have checked out the code from the CVS, instead of downloading a tarball. Try to check out the module to an empty directory, and then check the contents of the newly checked out
.infofile; the line is simply not there.Packaging script adds the into the file, and that's why it works for you—but, one should be able to maintain a Drupal installation directly from CVS.
Comment #7
danielb commentedoh thanks