This module throws a great many PHP notices when developing with E_ALL error reporting.

http://skitch.com/gregdunlap/ndg3k/casaa-settings-localhost

Much of this could be pretty easily cleaned up and it often uncovers some subtle bugs. Usually I would just write these patches but I'm kind of crunched for time and wanted to get it on your radar.

Comments

cntlscrut’s picture

cntlscrut commented
Assigned: Unassigned » cntlscrut

Thanks for pointing out those errors. I'll begin to work on cleaning those up as I can. As everything won't be fixed all at once I will update this issue as things are fixed and relate them to the respective future patch releases.

Once again, thanks.

cntlscrut’s picture

cntlscrut commented
Status: Active » Closed (won't fix)

Many of the php notices and warnings have been noted to clean up. Full compliance with E_ALL can be worked out but, it should not be an issue for clean code.