I just installed the module and, upon going to the configuration page, I got 3 messages basically telling me to read the INSTALL.txt and README.txt files.

Handily, they contain links for the files in the module's folder. The problem is the links appear to be generated relatively... This patch solves this by prefixing a slash to the !url's.

CommentFileSizeAuthor
#1 domain-help-messages.patch3.77 KBnicholasthompson
domain-help-messages.patch3.73 KBnicholasthompson

Comments

nicholasthompson’s picture

nicholasthompson
Primary language English
commented
StatusFileSize
new domain-help-messages.patch3.77 KB

Just realized that it would be FAR more sensible to use base_path() rather than just a slash! New patch attached.

Note: This patch is generated against HEAD - not rc3... But there is no option to select HEAD from the Versions.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented

That's odd. drupal_get_path() is the preferred solution here, since it can find the exact location of the module files and should write the links appropriately.

I assume you are running Drupal in a subfolder?

Possibly the statement needs to be wrapped in url(), which I believe is the cleaner solution than appending base_path().

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Needs review » Reviewed & tested by the community

Patch confirmed. Using url() causes the links to go through index.php which is not desired.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Reviewed & tested by the community » Closed (fixed)

committed to HEAD.