
Originally Posted by
Chrome
Hi all
I've added a few changes to the Updater's class file... Here are the changes:
- Changed the way the sidebox switch works (more a cosmetic/elegance alteration than anything, really)
- Fixed the errors caused by the reset() and each() calls (I hope)... This error was my making
- Altered the error logging method... Here I owe Design75 an apology as I incorrectly thought it was an example of bad design, but rather it's (I think) throwError() might be a magic function that's called in the event of any error in the class, rather than explicitly... It took me a while to figure out quite why the files were being created... After I figured this out I realised Design75's intention to alert the user to any error that had happened... I've added a drink icon as a peace offering to Design75 and my unreserved apologies... I'm sorry I reacted without fully checking and considering the situation... The logger now creates a single file in the logs folder and appends errors there
All that said, I've attached a replacement file for anyone that's kind enough to test it for me... The file only contains the affected class in its folder structure... Here's what needs to be tested:
- The _runDeep var that switches the sidebox on and off... This var is set in the class constructor
- The noted errors about reset() and each() to make sure they're squashed
- The new error logger... Easiest test is to manually call the logger directly, e.g. $this->throwError("Test error logger"); in the constructor
- The _runSilent flag that controls whether errors should be logged... Check that the log file is being created and it's in the correct place... Run the error test several times to ensure content is appending and not overwriting (this shouldn't be a problem)... Var is set in the constructor
All changes are commented but if anyone has any questions, please feel free to post here
Many thanks to everyone that tests and, again, my sincere apologies to Design75 (I really don't know where my head was)
Cheers
Dan
Bookmarks