search

segment-boneyard / analytics-magento

[DEPRECATED] The hassle-free way to integrate analytics into any Magento store.
15 stars 19 forks source link

Commented Mage:Log lines to prevent system.log from flooding #72

Closed 7ochem closed 8 years ago

7ochem commented 9 years ago

After simply going through the order process I have a lot of lines in my var/log/system.log coming from the Segment Analytics module. I commented out these lines as they provide no relevant information while all operations run normally. These lines would be necessary for developing, testing and debugging only.

I also commented out writing the HTML output to the segment.log in Segment_Analytics_Model_Observer::logBlockHtml() for the same reasons and during this simple test run the segment.log was already over 60Kb in file size.

selection_045

Please check and accept my changes, @sperand-io / @ianstormtaylor

sperand-io commented 8 years ago

Oof. Thank you!

Will release pending quick discussions on #66 and #70 — would love to get all those fixes out in one release.