It is faster to compare a string with === '' to find out if it's empty
than to run strlen() on it.
The replacement rules are applied as follows:
* if (strlen($str))
=> if ((string)$str !== '')
* if (!is_string($str) || strlen($str) === 0)
=> if (!is_string($str) || $str === '')
* If it can be seen easily that $str is a string,
the typecast is omitted.
Resolves: #54091
Releases: 6.2
Change-Id: I59c5cbccea4f98b8f282377e6aa67d970859a457
Reviewed-on: https://review.typo3.org/27091
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
This patch replaces most of around 1600 occurrences of
intval() and every (integer) in the whole core.
Additionally it changes GeneralUtility::intExplode to use references
and typecasting as well.
Some occurrences of strstr() together with intval() have been replaced
with strpos() as well.
And some superfluous intval calls have been removed or reduced
to a single one i.e. for protected variables or before loops.
Patch updated after Core CGL decision from 30.01.2014
Resolves: #54265
Releases: 6.2
Change-Id: Iba57ffad1f4233ffa1a9f7d3ca5bfe9d2b53f1e8
Reviewed-on: https://review.typo3.org/26740
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
The frontend editing + admin panel needs some polishing.
Following changes are included:
General
=======
- Reformat code
- Use Sprite API which means using the correct icons
Admin Panel
==============
- Add missing translations
- Show "Simulate usergroup" only if usergroups are available
- Remove "Total page count" in Info Tab as information is wrong
(counting images + document is not the total page size)
Frontend Editing
=================
- Remove "edit_editNoPopup", "forceNoPopup", "editFormsOnPage"
for direct inline editing
- Remove setting "previewBorder" which is not needed and looks just ugly
Releases: 6.2
Resolves: #49640
Change-Id: I35f88bc4e8c138fc21b74c0448b2f3892ab61918
Reviewed-on: https://review.typo3.org/21859
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Make use of boolean value TRUE as second parameter in sL function.
Change-Id: I6d77affc4f3ff260f617ca67c773fc4ca2e481ce
Resolves: #51143
Releases: 6.2
Reviewed-on: https://review.typo3.org/23153
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
This catches most old empty class files from ext:cms. The rest
needs to be taken care of in single patches.
The patch also changes all old mentions of the old class names,
mostly comments are affected.
Change-Id: I2b8ebfad178001c7bc8def967c01a56ae113c029
Resolves: #47792
Related: #47756
Releases: 6.2
Reviewed-on: https://review.typo3.org/20381
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reduce the severity if an internal flag is set and the a backend user
is logged in.
Resolves: #43735
Related: #43506
Releases: 6.1, 6.0, 4.7, 4.5
Change-Id: Ifd78f67d38b87180ab60df05167974705ee90352
Reviewed-on: https://review.typo3.org/17020
Reviewed-by: Stefan Neufeind
Reviewed-by: Markus Klein
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert
Some core parts still mention t3lib classes after namespaces,
mostly in comments.
Resolves: #47138
Releases: 6.1
Change-Id: I0775fc10f8c901a02533e5bc0fe1c6aa2276758b
Reviewed-on: https://review.typo3.org/19803
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
The core still access xliff language files as .php or .xml,
although the files themselves are converted to .xlf since quite
some time. This could cause confusion.
This is a cleanup taken all .xlf files in the EXT:lang/.xlf
directory and run via a sed script (thanks to Stucki)
Resolves: #46940
Releases: 6.1
Change-Id: I427776b176ba2c23678689029eaf388dff90d0fa
Reviewed-on: https://review.typo3.org/19579
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
In this patch are also some fixes for files containing
a interface. Due the namespace change all interfaces
have the curly bracket not on the same line as the
interface name.
Change-Id: I64ba45de73693452d681ce7f018965968b11d2af
Resolves: #46370
Releases: 6.1
Reviewed-on: https://review.typo3.org/18984
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Since #20849, we can specify a reason why we call
GLOBALS['TSFE']->set_no_cache().
Provide a reason for all calls done in core.
Resolves: #43506
Related: #20849
Releases: 6.1
Change-Id: Id786a041ef5d9f1db49d644459a593aaf54b5018
Reviewed-on: http://review.typo3.org/16925
Reviewed-by: Stefan Neufeind
Reviewed-by: Dmitry Dulepov
Tested-by: Dmitry Dulepov
With namespaces the @package and @subpackage annotations
do not have a real benefit anymore and can be removed.
Change-Id: Id8f4718db0c1c51ae5143a3353dd5efb42dadf70
Resolves: #43019
Releases: 6.0
Reviewed-on: http://review.typo3.org/16474
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
The text version of edit_showFieldHelp is not supported since 4.5
anymore. This patch removes all references and dead code.
Resolves: #24623
Releases: 6.0
Change-Id: Idb915f4dbaba8ee0690c06fddcace4d0d4ad6813
Reviewed-on: http://review.typo3.org/14950
Reviewed-by: Philipp Gampe
Tested-by: Philipp Gampe
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reverts the split-up for the extension reedit:
Erroneous extKey was 'fe_edit'.
Also fixes minor CGL.
Change-Id: If727b7ffcefdd1fb917fc15d48b91e549f5b79b9
Releases: 6.0
Fixes: #40174
Related: #40095
Reviewed-on: http://review.typo3.org/14073
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
