New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Change colors for the new buttons. #1
Change colors for the new buttons. #1
Conversation
but maybe a css fix is better? PR is: joomla#5146 |
I'm OK with this except that, until now, there has been a hidden field with value |
We can't do this because it is a backwards compatibility break and what is probably more nagging me is that we have to translate the values for JLog again. I wasn't aware this was an existing feature and thus changing it would be more hassle than it is worth. The PLT has no issues keeping it as it is now. Let's close this one and merge joomla#5146. |
ok closing here. maybe we can add a code comment that we change the values with 4.0? @okonomiyaki3000 @roland-d |
Allow only numbers in the count field
Use token validation for com_installer update.ajax
Just some very smal CS fixes.
Fixing some very minor CS
Code style + Improvement
Just fixing the whitespace error
Fix the issues mentiored by Travis and JM
CodeStyle two whitespaces ;)
Making some database fields required
Shuffle around the php-encryption library setup
Allow to set an option to hide the Use default on helpsite field
Use IN instead of OR + code style
Update en-GB.tpl_beez3.sys.ini
Unit test loading behavior, don't explicitly import JFactory now
… (joomla#12290) * Cleanups, fixes and a bit of optimizations for site/components batch #1 - com_ajax - com_banners - com_config Note: This is a single commit bundling all types of changes, since PR joomla#12261 which had detailed commits, was rejected as a whole * Ch-Ch-Changes! * Removed the query changes * Fix for wrong merge resolve
…otes-in-administrator-no-1 Replace unnecessary double quotes in administrator #1
- removed senseless statements - one case correction - one loop optimization - removed a closing tag
Some improvements in tests #1:
* Fixed callable call in loop termination condition * Fixed non-optimal regular expression * Shortened syntax for applied operations * Don't use strlen() to check if string is empty. * Reverted preg changes. Look into those later again. * Replaced call_user_func_array() with call_user_func() * Merge unset() calls * Replaced old intval() call with modern typecasting * Optimize away substr() as index-based access * Replace some cases of substr() with strpos() * Replace some stristr() with stripos() * Replace is_null() with null === ... * Removed unnecessary ternary operators * Fix for Juri... * Revert some changes for ... PHP5.3 compatibility * Make travis happy .... * Make codesniffer happy... * DYC! * Optimize non-optimal if conditions. * codsniffer... * Inline one-time use variables * Codesniffer... * Flipped comparisons `null === ...` to Joomla Style `... === null` `null !== ...` to Joomla Style `... !== null` * Flipped comparisons `false !== ...` to Joomla Style `... !== falsel` * Fix removal of ternary operator. Forgot inversion. * More flipping of comparisons ;) * Removed another unnecessary set of ternary operators. * Codesniffer... * Type-safe comparisons on strings * More type-safe comparisons * Changes that occurred during PR discussion * Remove some unnecessary parentheses * Removed empty if-group and reversed condition * Corrections on errors during merge (Conflict resolving) * Missed one. * Remove a parenthesis * Changes according to reviewer's comment * Revert change because of failure * Changes according to reviewer's comment + a few more changes found last minute * reversed change, though according to the method's doc that value should have been an integer... * Hopefully fixed... * Hopefully fixed... * Re-apply some changes
Add missing string COM_PRIVACY_MSG_CONSENT_NO_CONSENT
Implement:
Include -> Green
Exclude -> Red
@okonomiyaki3000 @roland-d