?
avatar brianteeman
brianteeman
14 Mar 2017

do we need the debug option? It says this is to send a report on every page load? Is that of any use to us at all @mbabker or is this a remnant code block that should have been removed

screenshotr15-48-52

avatar brianteeman brianteeman - open - 14 Mar 2017
avatar joomla-cms-bot joomla-cms-bot - change - 14 Mar 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 14 Mar 2017
avatar mbabker
mbabker - comment - 14 Mar 2017

Guess it's easier than hacking around the code or database if you're doing something that needs to trigger a send on every request. But there's no practical use for it for all but like 3 or 4 people I'd guess.

avatar brianteeman
brianteeman - comment - 14 Mar 2017

Yeah thats what I was thinking - dont really see the need for this in core

avatar brianteeman
brianteeman - comment - 14 Mar 2017

Should I remove it?

avatar zero-24
zero-24 - comment - 14 Mar 2017
avatar mbabker
mbabker - comment - 14 Mar 2017

I'm impartial either way. On the one hand if you ever have to work on the plugin it's going to be a useful thing to have, on the other hand it's not something most anyone would ever have a legitimate use for.

avatar zero-24
zero-24 - comment - 14 Mar 2017

What about make it use JDEBUG? or something like we have in the update mail plugin? ;)

avatar brianteeman
brianteeman - comment - 14 Mar 2017

If you ever have to work on the plugin then the two or three people that will need to will be more than cpaable of adding the required code manually

avatar brianteeman
brianteeman - comment - 15 Mar 2017

Closed as we have a PR see #14634

avatar brianteeman brianteeman - change - 15 Mar 2017
Status New Closed
Closed_Date 0000-00-00 00:00:00 2017-03-15 10:46:42
Closed_By brianteeman
avatar brianteeman brianteeman - close - 15 Mar 2017

Add a Comment

Login with GitHub to post a comment