What an excellent idea! Nice one!

Thank you so much for being part of the "Blue Move"!!!! This is very much appreciated!

I hope, other developers will jump in and help as well!

BTW - could you share with others what changes were required, and what "best practices" did you use?

nice idea for a module and very much needed i think!

however i think with the new version of XOOPS you can use preloaders to avoid hacking core files!

Havent used this yet but i'm sure travis could step in and provide you with some more info on the matter! well done for this module mate. keep up the good work. :)

Yup, use preloader and realise 0.5 :)

I thought the idea with this module was far to good to let it die (and I could see the use for it myself), so I decided to update it. It turned out that in this specific case, it was quite easy.

The only thing breaking this module in 2.4.x was the inclusion of some classes that are now part of the core. Removing the declaration of those classes in the module took care of that. As for PHP5 compliance, no changes were needed.

I'm in no way an expert at this, but working with simple modules like this one and XoopsPoll made me realize that if you put the effort in you may find it's not so hard to update them (the original module developers did all the hard work, for which they deserve all the credit ). I just use common sense and regular tools as the XOOPS debug, NetBeans with CodeSniffer plugin, and Firebug.

Hmmm, thinking about it, it may be worth putting together a short article on how to update modules, where real experts (unlike me) can contribute tips as well.

Regarding preloads, I haven't really had time to look into those, i.e. what they do and how they work. Also, this is another area where we could use documentation (yes, I know we are all aware of the lack of documentation, so this is not to bitch and moan about that), as I wouldn't know where to go to find info on how preloads work in XOOPS. Actually, me being not extremely familiar with the XOOPS core as such, I would love some documentation on what's actually included in the core that can be valuable for me to use in modules.

Quote:
Yes, that would be very good!

Currently the only option is to look at the API, but it doesn't give you the real overview, unless you are REALLY good in reading code.

Maybe some of the experts could do it for us....

Hi Mazarin..TQ...


take a look of this thread

http://www.xoops.org/modules/newbb/vi ... at&order=ASC&type=&mode=0

example of using preloads

Doesn't work. Preferences don't save. Database stays blank.

Warning: Invalid argument supplied for foreach() in file /modules/startup/admin/admin.php line 55

Lines 54-62;

Sorry about that. A bit of over sanitizing snuck into the released file. I will arrange for the file to be replaced tomorrow, but if you want a quick fix, please change lines 47 and 48 to look like this:

Edit: I've also updated the SVN so you can download it from there if you don't want to wait.

NEVERMIND. It works now.

THANK YOU!!!! EXCELLENT MODULE!

Quote:And sanitation, with eg intval()?

Yes, the intval didn't go to well with the array (or whatever similar type) from the listbox in line 48. Line 47 could still keep the intval function I guess.

intval is for checking integer data. There are other functions and techniques for all other data. One part of the move to blue is having save modules, with no risk for XSS.

OK, my bad. Then I suggest we change the heading of this news article to remove any mentioning of "blue" and I will pay special attention never to use that phrase again if not 100% valid.

No, no, it still goes blue!
It is a long way and one have to make it step by step.

NP I'll fix it by tomorrow (didn't have time today).