User tests: Successful: Unsuccessful:
Like #2253 but for staging instead of master.
Basically, since JLog
now supports category exclusion instead of just inclusion, the logging feature in this plugin should also give that option. Exclusion is much more useful than inclusion in this case anyway because you probably want to see most logs but you will never want to see databasequery
logs and you might get tired of looking at deprecated
logs.
Labels |
Added:
?
|
Category | ⇒ | Plugins |
Thanks!
@okonomiyaki3000 i think we should use class="btn-group btn-group-yesno"
for the Include/Exclude Button.
Can we move the logging options to the logging file set?
https://github.com/joomla/joomla-cms/blob/staging/plugins/system/debug/debug.xml#L81-108
https://github.com/joomla/joomla-cms/blob/staging/plugins/system/debug/debug.xml#L168
Or should we do this with in a different PR?
if i add 'jcontroller' or 'controller' i still get a 'jcontroller.log.php'
Good point about btn-group.
If you want it in the logging field set, that's a different PR but not one that I'm going to make.
Yes, you have misunderstood what this is about. This has nothing to do with witing logs to files. This about displaying logs in the debug "console" or whatever it's called.
Yes, you have misunderstood what this is about. This has nothing to do with witing logs to files. This about displaying logs in the debug "console" or whatever it's called.
Ok Now @test success.
Please add the class="btn-group btn-group-yesno"
and this is good to go i think.
Multiple good tests setting to RTc - thanks
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5146.
Status | Pending | ⇒ | Ready to Commit |
Labels |
Added:
?
|
@comitter please let @okonomiyaki3000 time to add the class="btn-group btn-group-yesno"
to the include / exclude button.
@okonomiyaki3000 @zero-24 Looking at this PR, it was correct to suggest to use the button group for the Include/Exclude. However I would like to see the Include/exclude swapped. It looks weird that Include is red and exclude is green.
Can you have a look into this PR against this branche?: okonomiyaki3000#1 This should implement it.
but maybe a css fix is better?
I'd like to hear some opinions on @zero-24 's PR. I don't have a problem with it but, it may cause a change to expected behavior in some cases (I can go into more detail if it's not obvious). I don't think it's a big deal so, unless there are any objections, I'll accept the PR.
ok PR is closed based on comment by @roland-d to keep backwards compatible. but maybe we can add a comment that this should be changed with 4.0 @okonomiyaki3000 ?
Labels |
Added:
?
|
Status | Ready to Commit | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-01-19 21:31:37 |
Milestone |
Added: |
But there's actually no BC issue unless we insist on swapping the colors which, to me, is totally arbitrary. It's not really that 'include' is affirmative while 'exclude' is negative. They're just two different modes. We don't even need colors at all, this option could just be a select.
Leaving it as-is isn't a problem either is it? I can't find it but there was something said about selects only for large listings, and button groups for 2 or 3 options.
It's not really a problem to leave it as-is but 'exclude' mode is far more useful than 'include' mode.
@okonomiyaki3000 I think you missed something. The exclude option has been merged into the core already.
Really? Great! What are we talking about then? Just changing the values so the colors will be different? Then I'm done because I do not care about that at all.
Milestone |
Removed: |
Milestone |
Added: |
Milestone |
Added: |
Milestone |
Removed: |
Labels |
Removed:
?
|
Setup
Before
You will see logs for every category. Mostly
databasequery
anddeprecated
After
You can set categories to
exclude
and list the ones you don't want to see. This definitely includesdatabasequery
because the debug plugin is listing them all anyway and it might also includedeprecated
unless you're on a mission to fix all that deprecated code. What's left will be a short, sweet list of logs that you might actually care about.