I discovered today that my custom implementation of theme_status_messages wasn't being executed because the CSM module was overriding my function.

The attached patch allows CSM to function as before, but also call the original status messages theme function.

CommentFileSizeAuthor
#1 1153244-1-no-theme-override.patch2.03 KBhanoii
csm-theme_status_messages.patch1.26 KBezheidtmann

Comments

hanoii’s picture

hanoii
Primary language Spanish
Location 🇦🇷UTC-3
commented
StatusFileSize
new 1153244-1-no-theme-override.patch2.03 KB

I did a different approach and remove the theme override altogether and change the message on the init callback.

Might not be the best for performance, but to be honest, is more or less the same because the message are also themed on each page request.

jim0203’s picture

jim0203 commented
Version: 6.x-1.0-alpha4 » 6.x-1.x-dev
Status: Active » Postponed (maintainer needs more info)

Can you check this against the most recent dev version of the module, please? I'm finding that a lot of issues in the queue have been inadvertently fixed by the rewrite that makes Status Messages Alter a dependancy.

stefank’s picture

stefank commented
Issue summary: View changes
Status: Postponed (maintainer needs more info) » Closed (fixed)

Marking as fixed. Please feel free to reopen if there is still an issue.