Feature #1744
Cleaning if() conditions
100%
Description
Examples:
if ($_configuration['multiple_access_urls'] == true) { // should be if ($_configuration['multiple_access_urls']) { ... } if ($visual_code_is_used == true) { // should be if ($visual_code_is_used) { ... } if (api_is_session_admin() == true) { // should be if (api_is_session_admin()) { ... } if ($export == true && $name != false) { // should be if ($export && $name) { ... }
etc. There are many similar statements within the source.
Conditions like these may be simplified as it was shown in the examples, for all the source.
History
Updated by Ivan Tcholakov over 9 years ago
- Status changed from New to Assigned
- Assignee set to Ivan Tcholakov
- Target version set to 1.8.8 stable
Ivan Tcholakov wrote:
Examples:
[...]
etc. There are many similar statements within the source.
Conditions like these may be simplified as it was shown in the examples, for all the source.
Updated by Ivan Tcholakov over 9 years ago
12182:15d63bdede83 Feature #1744 - Cleaning logical conditions: if ($_configuration['multiple_access_urls'] == true) ------> if ($_configuration['multiple_access_urls'])
http://code.google.com/p/chamilo/source/detail?r=15d63bdede836cf460eb779f58ed642e17e561ab&repo=classic
Updated by Ivan Tcholakov over 9 years ago
- % Done changed from 0 to 80
12185:473f22693552 Feature #1744 - Cleaning logical conditions (4).
http://code.google.com/p/chamilo/source/detail?r=473f22693552743d8636ed7521afbfbf2429f131&repo=classic
12184:9ac5cf7f894b Feature #1744 - Cleaning logical conditions (3).
http://code.google.com/p/chamilo/source/detail?r=9ac5cf7f894baa086ac6bfcceeb7bc39fd73efba&repo=classic
12183:5df76d535336 Feature #1744 - Cleaning logical conditions (2).
http://code.google.com/p/chamilo/source/detail?r=5df76d535336dc038df3231cbea0e6c04e85107d&repo=classic
Updated by Ivan Tcholakov over 9 years ago
- Status changed from Assigned to Feature implemented
- % Done changed from 80 to 100
12186:360bed6ce667 Feature #1744 - Online editor, YouTube plugin: Correction of the returned results of a javascript function.
http://code.google.com/p/chamilo/source/detail?r=360bed6ce667dd4c7b3ac0e04b35937b5a08567c&repo=classic