attached patch updates the module to the new settings approach. tested and working perfectly. also removes a little whitespace...

Comments

hunmonk’s picture

StatusFileSize
new3.69 KB

realized this is now a menu callback, and the function name and doxygen need to be changed accordingly. attached patch corrects this as well...

Torenware’s picture

Do you have a link to a description of the preferred approach you are implementing?

this is the first I've heard of this.

Thanks,
Rob

hunmonk’s picture

gerhard killesreiter’s picture

I'll apply that patch once I start the 5.0 conversion.

pwolanin’s picture

Is there an issue or forum topic to coordinate the 5.0 conversion. I'm also willing to submit patches.

killes@www.drop.org’s picture

Status: Reviewed & tested by the community » Needs work

doesn't apply anymore :(

Update of event.module for 5.0 has started.

pwolanin’s picture

@killes - Are you going to be posting patches to this or another issue, and looking for help or doing it all yourself? ;-)

Robrecht Jacques’s picture

Status: Needs work » Needs review
StatusFileSize
new1.49 KB

Part of the previous post was implemented. Part was implemented wrong. Updated patch attached.

hunmonk’s picture

Status: Needs review » Closed (duplicate)