I've created an info file so that the v5 modules module finds the module to display for the administrator as specified in http://drupal.org/node/64279. Please find globalredirect.info attached.

Earnie
http://for-my-kids.com

CommentFileSizeAuthor
#11 globalredirect_2.info221 bytesTobias Maier
#5 globalredirect_1.info220 bytesAnonymous (not verified)
#2 globalredirect_0.info247 bytesAnonymous (not verified)
globalredirect.info227 bytesAnonymous (not verified)

Comments

nicholasthompson’s picture

Excelent - I'll review this when I get a chance and commit it. I haven't had a chance to try it in Drupal 5.0 yet!

Anonymous’s picture

Component: Miscellaneous » User interface
StatusFileSize
new247 bytes

Updating my original file to include the version info as has been suggested at http://drupal.org/node/87298.

nicholasthompson’s picture

Cheers - I'll have to remember to submit that in too...

Tobias Maier’s picture

Status: Active » Needs work

remove the package = "Global Redirect"
per http://drupal.org/node/101009

there is no reason to start a package for a single module...

Anonymous’s picture

StatusFileSize
new220 bytes

Updated: removed package declaration and quoted description value.

Anonymous’s picture

Status: Needs work » Needs review
nicholasthompson’s picture

Cheers - I'll add this into the next release. Is there any potential issues with including this in a 4.7 release too? (I'm still new to CVS so its quite likely it'll take me a few released to get this right)

Anonymous’s picture

As long as you don't change your hook_help implementation delivering the .info file in 4.7 is just a file type Drupal doesn't know about and so ignores it. In 5.0 the meta data for the module display all comes from the .info file and the data from the hook_help is ignored. At least I think that is what happens. For all I know .info may have been introduced in 4.7 and enforced in 5.0 but either way it should be a file that is deliverable in 4.7.

nicholasthompson’s picture

I seem to have lost permissions to download the info files! Lol... Oh well they aren't hard to make yourself...

niklp’s picture

Yeah... I've just used this successfully...

; $Id$
name = Global Redirect
description = Prevents duplicate pages by checking for page aliases

I suspect this will break releasemonitor.module if you're using that, but I wouldn't expect this "problem" to last for that long anyway.

Mind you, what's with the permissions on the patches??

Tobias Maier’s picture

StatusFileSize
new221 bytes

here comes an updated file.
the name has to be in quotation marks, too

Anonymous’s picture

That must be new. I tested the .info file before submitting it.

Tobias Maier’s picture

Anonymous’s picture

Yes, it is different. BTW, something I learned along the way is that you should quote all the strings in the info file. It helps PHP to parse it more correctly.

nicholasthompson’s picture

The info is now included in the 5.0 release... I hope!
http://drupal.org/cvs?commit=53130

nicholasthompson’s picture

Status: Needs review » Fixed

As far as I can see - the info file is now included with V5.x releases.

killes@www.drop.org’s picture

Status: Fixed » Closed (fixed)