URL Link to settings on message [#2444717]

Firstly... YAY!! I love this module!! I have a custom header that covers the system message.
Kudos for this and I love how I have a choice of placement.

This is minor and I didn't know how to categorize it. It's more of an FYI

After I enable the module, a pop-up message appears saying that it's been enabled and to go to the settings page .... but when I click on it, it takes me to my error 500 page, so I keep going back and clicking to see what's wrong and copied the link directly. When I copied and pasted, it still came up with the same 500 error page.

So I clicked on the mod's configuration link and it took me directly to the page. Here's why

The configuration link shows this /?q=admin/config/user-interface/better-messages

-- note the "/?q=admin" vs "/admin"

/?q=admin/config/user-interface/better-messages
/admin/config/user-interface/better-messages

It's minor and probably a clean URL thing?

Anyhoo... thanks again for this great module =)

Comment	File	Size	Author
#4	better_messages-Better_rendering_of_configuration_link-2444717-4.patch	673 bytes	aditya.ghan
#2	better_messages-Better_rendering_of_configuration_link_after_installing-2444717-10321587-7.patch	673 bytes	aditya.ghan

Comments

Comment #1

aditya.ghan commented 12 September 2015 at 06:32

Hey Kebz,

I checked this issue, I found that the settings link in the message is generated by the base_path() function and then followed by the actual configuration path, I replaced that function with the ever dependable url() function which handles the clean URLs properly. I'm attaching a patch, maybe this will help you

Thanks!
Aditya