- Feb 26, 2017
-
-
Anja Leichsenring authored
irre_tuturial test extension uses TCA setting 'localizeReferences', but that setting does not exist and has no meaning in TCA. Remove it. Resolves: #80015 Releases: master Change-Id: I421b392cb7dec2c9530d86f926fd0169efe99b82 Reviewed-on: https://review.typo3.org/51848 Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
TYPO3com <no-reply@typo3.com> Reviewed-by:
Thomas Hohn <thomas@hohn.dk> Tested-by:
Wouter Wolters <typo3@wouterwolters.nl> Tested-by:
-
Thomas Hohn authored
Join consecutive lines for INSERT statements in ext_tables_static+adt.sql using a space instead of a linefeed to avoid parser errors. Resolves: #78892 Releases: master Change-Id: Ide550419b7ef41109a660a671b8aba65a33eb6b3 Reviewed-on: https://review.typo3.org/51854 Reviewed-by:
Morton Jonuschat <m.jonuschat@mojocode.de> Tested-by:
Helmut Hummel <typo3@helhum.io> Tested-by:
-
Helmut Hummel authored
The dynamic variable syntax does currently work in Fluid standalone, but not with the variable provider used for TYPO3 CMS (as promised) To fix this, we copy the necessary code to the CMS variable provider as it is only a few lines. Cover that with tests and apply it in the image cropper UI as one example. Resolves: #79997 Releases: master Change-Id: I3a811b0d496b12191d2ccbaa9c07525f4cf8f340 Reviewed-on: https://review.typo3.org/51841 Tested-by:
Anders Kostending <aha@systime.dk> Reviewed-by:
Georg Ringer <georg.ringer@gmail.com> Tested-by:
Joerg Boesche <typo3@joergboesche.de> Reviewed-by:
-
- Feb 25, 2017
-
-
Susanne Moog authored
The current user permissions were not initialized for the context menu resulting in non-existing permissions for editors (and no possibility to copy a page). Change-Id: I8fe747a1f19b4bea794b7fdbd676e7f366eea695 Resolves: #80019 Releases: master Reviewed-on: https://review.typo3.org/51852 Tested-by:
Tymoteusz Motylewski <t.motylewski@gmail.com> Tested-by:
-
Sascha Egerer authored
Throw a meaningful InvalidXmlFileException when an invalid locallang.xml file is detected. Resolves: #79908 Releases: master, 7.6 Change-Id: Idc5586166d2475edeaeef407128525ea57fb9a09 Reviewed-on: https://review.typo3.org/51761 Reviewed-by:
Stephan Großberndt <stephan@grossberndt.de> Tested-by:
Faton Haliti <fha@systime.dk> Tested-by:
-
Mads Jensen authored
Fixes a bug where editing a backend user record without updating the password would result in the password being set to the string literal "*********". This reverts #79576 because the fix for showing the password hash in the readable field was wrong and causing this bug. Instead of forcing the database value in the hidden field to be asterisks, we now correctly set the type of the human readable field to be password. This triggers a special handling in the form engine JavaScript, not filling the human readable field with the database value and switching to type text when entering a new password. Resolves: #79714 Reverts: #79576 Releases: master Change-Id: Ia465293272131c32bbb9fd9b0d3916676e130996 Reviewed-on: https://review.typo3.org/51829 Reviewed-by:
Nicole Cordes <typo3@cordes.co> Tested-by:
Mads Lønne Jensen <mlj@systime.dk> Tested-by:
-
Christer Vindberg authored
Moved the check for if the target is coded as a JS open window link, to a later stage in the function. Since Css Styled Content and Fluid Styled Content was refactored, the target might not be set until later in the typoLink function. This caused pop links to stop working, moving the check for a pop up link target to a later stages fixes the problem. Resolves: #79918 Releases: master Change-Id: Ie27fa3142df83c33ba3ad465ecb35c0f6e571f86 Reviewed-on: https://review.typo3.org/51830 Tested-by:
Jasmina Ließmann <code@frauliessmann.de> Tested-by:
Timo Hund <timo.hund@yahoo.com> Tested-by:
-
Nicole Cordes authored
The current implementation for getting paginated items from ObjectStorage objects in TYPO3\CMS\Fluid\ViewHelpers\Widget\Controller\PaginateController enforces null items if the count of objects is lower than the itemsPerPage count. This patch calculates the correct count and returns valid items only. Resolves: #79559 Releases: master, 7.6 Change-Id: I5746d83120474d6246da949f67096459ed1076d7 Reviewed-on: https://review.typo3.org/51475 Tested-by:
-
Nicole Cordes authored
Due to some security changes in GeneralUtility::validPathStr some wrongly encoded files are now returned as invalid. This breaks e.g. the file list. This patch adds a try/catch block to ignore invalid files and return the valid ones. Resolves: #79776 Releases: master, 7.6, 6.2 Change-Id: Iaa3697b04c123bba61a426fdf39ee3959f68b705 Reviewed-on: https://review.typo3.org/51655 Tested-by:
Tobias Liebig <tobias.liebig@typo3.org> Tested-by:
Sascha Egerer <sascha@sascha-egerer.de> Tested-by:
-
Raphael Okon authored
The fix is to remove previously introduced CSS hacks and provide a cleaner and leaner base Resolves: #79998 Resolves: #79989 Releases: master Change-Id: Ic63b3acaf46f50f92a3c44578c28748dfca74351 Reviewed-on: https://review.typo3.org/51842 Tested-by:
-
- Feb 24, 2017
-
-
jliessmann authored
Use bootstrap grid to arrange image thumbnails Resolves: #79960 Releases: master Change-Id: I99ef6fc159c1497ea5464ef33398cf799c91a534 Reviewed-on: https://review.typo3.org/51810 Reviewed-by:
Joerg Kummer <service@enobe.de> Tested-by:
-
Christer Vindberg authored
When [FE][pageNotFoundOnCHashError] = true, searching with Indexed search in FE causes an 404, because the POST action is missing the cHash. Resolves: #79970 Releases: master Change-Id: If4f205ddfd264b1213ef696265fda0a8a6d9f29c Reviewed-on: https://review.typo3.org/51809 Tested-by:
-
Nicole Cordes authored
This patch adds the modal context to be able to confirm the installation of an incompatible extension. Resolves: #79995 Releases: master Change-Id: I84187076684eb1fb4ee69679f8003ec115a6bf2c Reviewed-on: https://review.typo3.org/51826 Tested-by:
-
Thomas Hohn authored
Some references to the _cli_lowlevel were still present and have been removed. Resolves: #79986 Releases: master Change-Id: I79473cc422a21761b3474facb4f981501ed7f545 Reviewed-on: https://review.typo3.org/51814 Reviewed-by:
Josef Glatz <josef.glatz@typo3.org> Tested-by:
Christer V <cvi@systime.dk> Tested-by:
-
Mads Jensen authored
Added CSS to ensure images are contained in parent element. Resolves: #79971 Releases: master Change-Id: I38a4038af067166d771d012e2ddc0e2983060513 Reviewed-on: https://review.typo3.org/51802 Reviewed-by:
Morten Pless <mpl@systime.dk> Tested-by:
-
Timo Schmidt authored
Currently you get an error in the backend when you try to set the language to "all" for a new or existing content element (integer was expected but null received triggers PHP strict type error). The reason for this is, that the DataMapProcessor tries to sychronize elements from tt_content with uid 0. This patch adds an integration test that content elements without an existing source do not get synronized. Resolves: #79853 Releases: master Change-Id: I67d4b35efe1564c13e9faf085c4848f2d0313998 Reviewed-on: https://review.typo3.org/51804 Tested-by:
Claus Due <claus@phpmind.net> Tested-by:
Johannes Goslar <jogo@kronberger-spiele.de> Reviewed-by:
-
Christian Kuhn authored
A recently added test in extbase query building reveals a non-trivial-to-fix issue of the created query with postgres. This is the last issue we have with functionals on postgres, to have them running as pre-merge tests we for now disable the offending tests on that database platform, and handle the real fix in a separate issue. Change-Id: I830af04ecdb41fa968421521e9860c388f74d75b Resolves: #79987 Related: #79988 Related: #79931 Related: #79932 Related: #79330 Releases: master Reviewed-on: https://review.typo3.org/51818 Reviewed-by:
Mona Muzaffar <mona.muzaffar@gmx.de> Tested-by:
-
Morton Jonuschat authored
Deprecate BackendUtility::getRecordsByField() as it has a flawed design due to passing SQL fragments. This contradicts the goal of using named parameters for all queries in the core and requires passing the original QueryBuilder object in addition to the stringified constraint. Replace all calls to the method with direct usage of the QueryBuilder and deprecate the method. Resolves: #79122 Releases: master Change-Id: I8b040b98e20271aff84ef16fb89b59a406d54003 Reviewed-on: https://review.typo3.org/51078 Tested-by:
-
Benjamin Franzke authored
All redirect status codes (e.g. 201,301,303,307) are incorrectly sent as 302. We need to set the response code before setting the Location header, as a call to header('Location: ...'); sets the response code implicitly to 302 (unless it has already been set to 201 or 3xx) [1]. By settings the status code/header first, header('Location: ..') will not change the status code, if the PSR-7 response object contains a proper redirect status code. [1] http://php.net/manual/en/function.header.php The second special case is the "Location:" header. Not only does it send this header back to the browser, but it also returns a REDIRECT (302) status code to the browser unless the 201 or a 3xx status code has already been set. Change-Id: Ifa076e6393c6ed42d93959fe8c3a5b79cee145a4 Fixes: #79043 Releases: master, 7.6 Reviewed-on: https://review.typo3.org/51002 Tested-by:Andreas Fernandez <typo3@scripting-base.de> Reviewed-by:
Philipp Gampe <philipp.gampe@typo3.org> Tested-by:
Marco Huber <mail@marco-huber.de> Tested-by:
Susanne Moog <susanne.moog@typo3.org> Tested-by:
-
Josef Glatz authored
Add a description field also for fe_users records which could be used as for already known other record types. Resolves: #79894 Releases: master Change-Id: Ia1d7ad7746a0b7dd3ebb809864e290db97ac228d Reviewed-on: https://review.typo3.org/51745 Tested-by:
Riccardo De Contardi <erredeco@gmail.com> Reviewed-by:
Benni Mack <benni@typo3.org> Tested-by:
-
Jasmina Liessmann authored
Use Bootstrap grid for arrangement of search form fields. Resolves: #79848 Releases: master Change-Id: Ied1b30a2ba81b339ac93a9a874720b43623e6e66 Reviewed-on: https://review.typo3.org/51812 Reviewed-by:
-
- Feb 23, 2017
-
-
Helmut Hummel authored
Fix the changed path as well Resolves: #79980 Releases: master Change-Id: I5b23e4bb56f47d151b506439c4e46b9c0838bd87 Reviewed-on: https://review.typo3.org/51811 Reviewed-by:
Frank Nägler <frank.naegler@typo3.org> Tested-by:
-
Morton Jonuschat authored
Introduce dedicated fixures for the export data based on the database platform being tested. Due to mysqlnd and PDO based drivers returning different types of values (mainly int vs string) the resulting XML is dependent on the database. Change-Id: I776cfbc28c7355e72c21fa89097077f94225e223 Resolves: #79798 Releases: master Reviewed-on: https://review.typo3.org/51774 Reviewed-by:
Manuel Selbach <manuel_selbach@yahoo.de> Tested-by:
-
Christian Futterlieb authored
In order to avoid time-based hash-based attacks, the native PHP security functions are used instead of simple string comparisons, when comparing passwords with hashes. Change-Id: I0dbe2c12c5017f9d71ea7628ddd35d919510ac12 Releases: master Resolves: #79888 Related: #79795 Reviewed-on: https://review.typo3.org/51737 Reviewed-by:
Markus Klein <markus.klein@typo3.org> Tested-by:
-
Thomas Hohn authored
During copying of pagetrees various sql statments are executed. One on sys_file_reference has a bad performance and can benefit from an index. Fixes: #79604 Releases: master Change-Id: I2caa83db6259ad6baad75427b5040a11ece768ba Reviewed-on: https://review.typo3.org/51512 Tested-by:
-
Sascha Nowak authored
In the current implementation it is not possible to create a new translations outside of the default translation. So we can readd the new translation option when the record has a default translation. Change-Id: I89c4200ff847dadfdeec03b5f43a6577f40702e3 Resolves: #72797 Releases: master, 7.6 Reviewed-on: https://review.typo3.org/51092 Tested-by:
Alexander Opitz <opitz.alexander@googlemail.com> Tested-by:
-
Stefan Froemken authored
It is not possible to change or override cropVariants via TSconfig for sys_file_reference (e.g. for a specific user or page tree branch), thus making the cropping functionality a pain for bigger sites. Setting a FormEngine option as expected like TCEFORM.sys_file_reference.crop.config.cropVariants.* does now work. Resolves: #79812 Releases: master Change-Id: I9d36f6661c2a5ca051b803f06485fdb4c8c8a437 Reviewed-on: https://review.typo3.org/51676 Reviewed-by:
Frans Saris <franssaris@gmail.com> Tested-by:
-
Sascha Egerer authored
* Ensure that variables are correctly initalized. * Fix some php docs. * Initalizes for loop iterator variables before the loop if they are also used after the for loop is finished. * Remove superfluous function arguments. Resolves: #79892 Releases: master Change-Id: I75109b2914f2b05806a303837b7aad79be8e3707 Reviewed-on: https://review.typo3.org/51742 Reviewed-by:
-
Christian Kuhn authored
Change-Id: I89f5e73ab896d6cd4dac46ebc9e64e14643adbeb Resolves: #79725 Releases: master Reviewed-on: https://review.typo3.org/51800 Tested-by:
Anja Leichsenring <aleichsenring@ab-softlab.de> Tested-by:
-
Mona Muzaffar authored
valuePicker appends and prepends now in the correct order. Change-Id: Ic97b8a66469d43d9269f67f1de3f0cb40d1e8321 Resolves: #79937 Releases: master, 7.6 Reviewed-on: https://review.typo3.org/51779 Tested-by:
-
Frans Saris authored
The introduction of the new crop variants #75880 broke the handling of cropped images when using typoscript to render file(reference)'s This patch fixes the rendering of cropped images and introduces the new cropVariant option to the typoscript object IMG_RESOURCE Releated: #75880 Resolves: #79883 Releases: master Change-Id: Icba77d76c6914d48dcc57d7e8c48471ed27c44b3 Reviewed-on: https://review.typo3.org/51753 Reviewed-by:
-
Mona Muzaffar authored
Skip tests downloadALanguage() and activateAndDeactivateALanguage() Tests need to be fixed. Change-Id: I999359a83e0d8e092fda483a7799c70110eb10e4 Resolves: #79962 Releases: master Reviewed-on: https://review.typo3.org/51798 Tested-by:
-
Oliver Hader authored
In case TCA field values have not been migrated yet, but their TCA at the same time already being updated to allowLanguageSynchronization, then the field values are not considered in the upgrade wizard. This change now includes manually updated TCA settings as well. Resolves: #79950 Related: #79658 Releases: master Change-Id: Id239c569e3731691a54015642e56a6450c09badc Reviewed-on: https://review.typo3.org/51692 Tested-by:
-
- Feb 22, 2017
-
-
Benni Mack authored
There are still some properties and additional method call arguments that are not evaluated properly anymore and can be omitted. Resolves: #79959 Releases: master Change-Id: Ifc4ed71891394af4c80eaa9c4bb987f79c980a73 Reviewed-on: https://review.typo3.org/51793 Tested-by:
-
Claus Due authored
Solves three issues: * Namespace declaration output in templates without sections * Potential error from unexpected input argument in sanitizePath * Rendering of new HeaderAssets / FooterAssets sections in uncompiled templates. Adds two Fluid features: * Namespace detection is now in sync with TYPO3 CMS. * Escaping modifier `{escaping off}` now supported. And allows the removal of two custom Fluid integrations contained in TYPO3 which are now included in Fluid. Change-Id: Ic9cd73d1d05688139241c81e6a3fb78998fe9407 Resolves: #79847 Resolves: #79811 Releases: master Reviewed-on: https://review.typo3.org/51711 Tested-by: -
Benni Mack authored
Streamlined EXT:about extension as a first step to clean up the whole TYPO3 Core base: - Moved locallang file to EXT:about, as it is only used in this extension - Removed usage of TYPO3_LOADED_EXT/ext_emconf.php in favor of PackageManager (needed to add the authors into each Package) - Fully added all external libraries in use (not including -dev requirements) - Moved EqualHeight.js to EXT:about as it is only used in this extension - Proper usage of Fluid layouts - Streamlined code, variables, imports and comments - Reviewed comments in PHP Resolves: #79948 Releases: master Change-Id: Ia6c2d6b519bc34847131f06908c2c441b9248a8b Reviewed-on: https://review.typo3.org/51790 Tested-by:
-
Christian Kuhn authored
The ImageCropUpdater of the install tool does not need the not-yet-migrated TCA it does not rely on any TCA that may have changed. Remove that section from the code. Change-Id: I8ed4190fd53d87f368c978b183b99b562dba9444 Resolves: #79951 Releases: master Reviewed-on: https://review.typo3.org/51792 Reviewed-by:
-
Georg Ringer authored
- Use a div instead of an input field to avoid issues like tooltip not shown because of the disabled attribute. - Trigger onchange from LinkBrowser popup - If no value found, show the input field, which makes it easier to e.g. enter an external url. - Show helpful title for toggle button on hover Resolves: #79667 Related: #79670 Releases: master Change-Id: If00d892d0ae7aebe0dfc5f17f5d43af16d0ee730 Reviewed-on: https://review.typo3.org/51592 Reviewed-by:
-
- Feb 21, 2017
-
-
Christian Kuhn authored
Fix for issue #78628 fixed "addItems" for TCA trees, but broke them for select drop downs. Juggle icon providing around a bit to fix. Change-Id: I3a1077992a7db0d72bc2d7d0247292763cc25958 Resolves: #78747 Related: #78628 Releases: master Reviewed-on: https://review.typo3.org/51766 Tested-by:
-
Johannes Kasberger authored
This change fixes a regression after the doctrine migration and additionally fixes a wrong join condition which is also present in version 7 Resolves: #79931 Resolves: #79932 Releases: master Change-Id: I04a4ce174fb1da3baca9af8ba771a7db70d9a884 Reviewed-on: https://review.typo3.org/51782 Tested-by:
-
