- Feb 16, 2019
-
-
Andreas Fernandez authored
When the opendocs toolbar item gets initialized, the document count shows a bogus value as deleted records are not removed from the calculation. As the documents are loaded deferred since #84412, the correct value gets in place automatically which renders the initial counting obsolete and is removed with this patch. Additonally, a wrong class hint in OpenDocumentController is fixed. Resolves: #87725 Related: #84412 Releases: master, 9.5 Change-Id: I0684e0950d0f4daf11042d6c078b5598bfb3d4be Reviewed-on: https://review.typo3.org/59708 Tested-by:
TYPO3com <noreply@typo3.com> Reviewed-by:
Anja Leichsenring <aleichsenring@ab-softlab.de> Tested-by:
Georg Ringer <georg.ringer@gmail.com> Tested-by:
Björn Jacob <bjoern.jacob@tritum.de> Reviewed-by:
Tymoteusz Motylewski <t.motylewski@gmail.com> Tested-by:
-
- Feb 15, 2019
-
-
Benni Mack authored
When building TypoScript like this: 10 = TEXT 10.value = English 10.lang.dk = danish This does not work currently as it relies on "config.language". This part has been forgotten while the Site Handling was initially implemented into the TYPO3 core. The patch adapts the changes to make use of Site Handling if it is available, but falls back to the TypoScript options (for PseudoSites). Resolves: #87712 Releases: master, 9.5 Change-Id: I5e2b97b178e6ccafec1ff7556d41321ea67fb45a Reviewed-on: https://review.typo3.org/59698 Tested-by:
Daniel Goerz <daniel.goerz@posteo.de> Tested-by:
Jörg Bösche <typo3@joergboesche.de> Reviewed-by:
Josef Glatz <josef.glatz@typo3.org> Tested-by:
-
- Feb 14, 2019
-
-
Sybille Peters authored
Resolves: #85461 Releases: master, 9.5, 8.7 Change-Id: I36eef1d24fce233639e2047cc2f09d972ccbe046 Reviewed-on: https://review.typo3.org/59627 Reviewed-by:
Benni Mack <benni@typo3.org> Tested-by:
-
Julian Wundrak authored
Add check option to site-config tca for something like active or maintenance mode Resolves: #87692 Releases: master, 9.5 Change-Id: Idaa09a5929db5bdf177e57c5f78f827047705f91 Reviewed-on: https://review.typo3.org/59683 Tested-by:
André Schließer <andy.schliesser@gmail.com> Tested-by:
-
- Feb 13, 2019
-
-
Frank Naegler authored
The AdminPanel Cache Module has to implement the ResourceProviderInterface to load the required JavaScript module. Resolves: #87701 Releases: master, 9.5 Change-Id: Ib8c440bc5847da64aad480dc98adda7b440f4bf9 Reviewed-on: https://review.typo3.org/59688 Reviewed-by:
Andreas Fernandez <a.fernandez@scripting-base.de> Tested-by:
Markus Klein <markus.klein@typo3.org> Tested-by:
-
Richard Haeser authored
To prevent type mismatches when using pdo_mysql as the database driver, the data for the lastMod field is cast to an integer. Resolves: #87689 Releases: master, 9.5 Change-Id: I696f62a3b08d464708c9e1ab9edb4c733835f475 Reviewed-on: https://review.typo3.org/59682 Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Tested-by:
Guido Schmechel <guido.schmechel@brandung.de> Reviewed-by:
-
- Feb 12, 2019
-
-
Stefan Froemken authored
Resolves: #87696 Releases: master Change-Id: I50eb442bc2130ed5a0741ea3f71558257535ff89 Reviewed-on: https://review.typo3.org/59685 Reviewed-by:
Pascal Rinker <info@crynton.com> Tested-by:
-
- Feb 11, 2019
-
-
Benni Mack authored
Used * composer update friendsofphp/php-cs-fixer --with-all-dependencies The new php-cs-fixer is compatible with PHP7.3+ and fixes some more issues configured according to our code style settings. Resolves: #87684 Releases: master, 9.5 Change-Id: Ibb2150eea173ded11495866aa171dd171aca7efd Reviewed-on: https://review.typo3.org/59677 Reviewed-by:
-
- Feb 09, 2019
-
-
Benni Mack authored
doctrine/dbal 2.7.x will reach end of life before core v9 is out of support. We should upgrade to at least 2.8: composer req doctrine/dbal:~2.8.0 https://www.doctrine-project.org/2018/07/12/common-2-9-and-dbal-2-8-and-orm-2-6-2.html Also needs a typo3/testing-framework raise: composer req --dev typo3/testing-framework:~5.0.5 Needed core changes: - DBAL does not add "OFFSET 0" if nothing given (test change only) - SQLite does not need "PRIMARY KEY" if autoincrement is given (in DefaultTcaSchema) Resolves: #86490 Releases: master, 9.5 Change-Id: I28014d252a57f7f421242e35086c63a0a824ed04 Reviewed-on: https://review.typo3.org/59668 Tested-by:
-
- Feb 08, 2019
-
-
Oliver Hader authored
Having 1:n child associations being defined as l10n_mode=exclude on their parent side, currently leads to deleting and recreating the child entity (re-synchronization). For composite relations (children can only exist with their parent - usually accessed through their parent as aggregate root) this is "okay" in terms of domain-driven design. However having large data-sets leads to performance impact during the re-synchronization process. In the current scenario children processed with l10n_mode=exclude did not have any pointer to their language origin (due to l10n_parent not being set). This change copies these children and applies the same values as used for localizations - without actually invoking the localization process. For l10n_mode=exclude children this means, the sys_language_uid and l10n_parent values are now set - which have been empty before. Resolves: #87640 Releases: master, 9.5 Change-Id: I3d862f536603b9e49c7a5d3205ccfc2b4e2e9532 Reviewed-on: https://review.typo3.org/59629 Tested-by:
-
Georg Ringer authored
To allow more usecases for extending the site configuration, the value of tca type select should only be typecasted if it looks like an integer. Resolves: #87663 Releases: master, 9.5 Change-Id: I700bba3ac67ceac9acc8aa473d1b723b4bc34f4f Reviewed-on: https://review.typo3.org/59649 Tested-by:
-
Thorben Nissen authored
Remove the label `Check/Uncheck all` from the progress bar and show only the process percentage. Resolves: #87624 Releases: master, 9.5 Change-Id: Iff6cbb47099747f613d4ea90ab75863454c2669b Reviewed-on: https://review.typo3.org/59621 Reviewed-by:
-
Andreas Fernandez authored
To prepare a couple of bugfixes regarding the language fallback handling, the property `SiteLanguage::$fallbackLanguageIds` must be an array of integer values. The patch ensures that all values are cast to integers. Resolves: #87673 Releases: master, 9.5 Change-Id: I437a08ce992718f601a612df4cddd9ecac5e1d3a Reviewed-on: https://review.typo3.org/59662 Reviewed-by:
Alexander Schnitzler <typo3@alexanderschnitzler.de> Tested-by:
-
- Feb 07, 2019
-
-
Benni Mack authored
All PSR-15 middlewares now do not set the $GLOBALS['TYPO3_REQUEST'] object anymore. Instead, the RequestHandlers are using them for all other logic (e.g. GP etc) where the request object or any other attribute has not been handed in yet. It is now safe to say that PSR-7 request/response objects have to be used within middlewares to access or manipulate GET/POST data. Also, the safety net for overriding GET/POST again is still in place for FE (data = GP:id) in order for existing TypoScript to work still, until all other code gets global-free. There are still places in TYPO3 Core where $TYPO3_REQUEST is necessary (TypoScriptConditionMatcher, Error Handling) which are added accordingly. Resolves: #87661 Releases: master Change-Id: Ibe499b6fda86ccd3abefcb3c8be294a7cb765d74 Reviewed-on: https://review.typo3.org/59490 Reviewed-by:
-
Benni Mack authored
The accepted change adds an ugly side-effect when using the same controller/action in multiple route variants. This reverts commit 977ed578. Resolves: #87668 Reverts: #87337 Releases: master, 9.5 Change-Id: I7167b42cb26538e676256c9d2e36d1ec18f2bd40 Reviewed-on: https://review.typo3.org/59658 Tested-by:
Oliver Hader <oliver.hader@typo3.org> Tested-by:
-
Guido Schmechel authored
If the controller of a route in a routeEnhancer differs from the default controller, it will now be used for the generation of enhancers. Resolves: #87337 Releases: master, 9.5 Change-Id: Iea3d15bbc99cb17c640e6a4824d3dd42176295d1 Reviewed-on: https://review.typo3.org/59343 Tested-by:
Alexander Grein <alexander.grein@gmail.com> Tested-by:
Richard Haeser <richard@maxserv.com> Reviewed-by:
-
Georg Ringer authored
InputOption::VALUE_REQUIRED must be used instead of InputArgument::REQUIRED. Resolves: #87648 Releases: master, 9.5 Change-Id: Ifd727ec26af0e1e2237e9ca5ae480b9ed2fd5b6f Reviewed-on: https://review.typo3.org/59643 Tested-by:
-
- Feb 06, 2019
-
-
Oliver Hader authored
Adds DataHandler tests that were available for TCA type inline/FF for inline/CSV as well (initially introduced with issue ##79856). Resolves: #87666 Releases: master, 9.5, 8.7 Change-Id: I01eadae3fc00a0ba037e1c8ef7e92b6de467266d Reviewed-on: https://review.typo3.org/59652 Tested-by:
-
Alexander Schnitzler authored
Class QueryFactory does no longer use setter injection but constructor injection instead. Releases: master Resolves: #87642 Change-Id: I5d595ff2c9b5b7e81f5d6380eef1284e0e2d9739 Reviewed-on: https://review.typo3.org/59631 Tested-by:
Mathias Brodala <mbrodala@pagemachine.de> Tested-by:
-
Benni Mack authored
The logic within EXT:felogin for validating proper redirect/referer URLs is extracted into a new PHP class, which can now be called separately. This is useful for further refactorings within the Controller, and it lays the foundation to use proper validation in various places other than Extbase. Resolves: #87660 Releases: master Change-Id: I79aab5908bde869b7ee3cf730fbf61d658f2ee88 Reviewed-on: https://review.typo3.org/59648 Tested-by:
-
- Feb 05, 2019
-
-
Josef Glatz authored
All occurrences of APC (without suffix "u") are removed from the TYPO3 core since APC is discontinued and does not work with PHP 7.x. This changes does not affect the PHP OPcache related area. Releases: master Resolves: #87583 Change-Id: Ie724f8343dae9bdc4721d35c6d384c1de51f365e Reviewed-on: https://review.typo3.org/59588 Tested-by:
Alexander Opitz <opitz.alexander@googlemail.com> Tested-by:
-
Stephan Großberndt authored
When editing a fe_group show its subgroups even if those are set to hidden in order to avoid losing the subgroup relation on saving the fe_group. Resolves: #87649 Releases: master, 9.5 Change-Id: Ic39f32b47327202a848827bd9d6b4935c35edcdd Reviewed-on: https://review.typo3.org/59640 Tested-by:
Xavier Perseguers <xavier@typo3.org> Tested-by:
Andreas Wolf <andreas.wolf@typo3.org> Tested-by:
-
Alexander Schnitzler authored
ObjectAccess had two kinds of runtime reflection calls: 1) To gather data about object properties and methods 2) To make non public properties accessible The first one is tackled by using class schema instances and by using the property accessor of symfony/property-access. The latter is tackled by deprecating all method arguments that trigger reflection to make properties accessible. In the future, gettable/settable properties need to be either public or have a getter (get*/has*/is*) or setter. Releases: master Resolves: #87332 Change-Id: I6ecef81de7aa4cc1244166d683874a1a87ac6bb7 Reviewed-on: https://review.typo3.org/59331 Reviewed-by:
-
- Feb 03, 2019
-
-
Alexander Schnitzler authored
This patch introduces the requirement to symfony/property-info which provides a neat api to extract information about properties via several different extractors. The package comes with a PhpDocExtractor, which is kind of a drop in replacement (functional wise) for the extbase DocCommentParser which has been removed. Along with the replacement of the doc block extractor the package comes with an api to fetch context data that enables us to resolve non fully qualified class names. This is now possible: ``` use TYPO3\CMS\Extbase\Persistence\ObjectStorage; use ExtbaseTeam\BlogExample\Domain\Model\Comment; class Post { /* * @var ObjectStorage<Comment> */ public $comments; } ``` Important: This only works in extbase models as the reflection costs are high and the information is only needed in this case. The non fully qualified class name is now also supported for injection properties, although it is still recommended to avoid injection properties in favor of injection methods or constructor injection. Example: ``` use TYPO3\CMS\Extbase\Annotation as Extbase; use TYPO3\CMS\Extbase\Configuration\ConfigurationManager; class Service { /* * @Extbase\Inject * @var ConfigurationManager */ public $configurationManager; } ``` Releases: master Resolves: #87457 Change-Id: I006aeb737b4bb9790fda257e12ba46a105d9b113 Reviewed-on: https://review.typo3.org/59454 Tested-by:
-
- Feb 02, 2019
-
-
Benjamin Franzke authored
The recently introduced `object` type declarations for extbase classes require a minimum version of prophecy 1.7.5. Previous versions trigger the following error: Scalar type declaration 'object' must be unqualified Prophecy added support for object type declarations in 1.7.4 and for return type object declarations in 1.7.5. Therefore we add an explicit dev dependency to prophecy in composer.json now (previously it was transitively required as ^1.7 by phpunit 7.1). composer require --no-update --dev phpspec/prophecy:^1.7.5 composer update --lock Releases: master Resolves: #87622 Change-Id: Ic35db424d1ea46143839ec3f4bd3c10e39c99b4e Reviewed-on: https://review.typo3.org/59620 Reviewed-by:
-
- Feb 01, 2019
-
-
Markus Klösges authored
When saving and retrieving object from the persistence session, implementation class names set via \TYPO3\CMS\Extbase\Object\Container are now respected by storing and retrieving them with that overridden class name. Resolves: #77617 Releases: master, 9.5 Change-Id: I887cb164328afe789dc274c92c8f8774b2098a40 Reviewed-on: https://review.typo3.org/58959 Tested-by:
Mona Muzaffar <mona.muzaffar@gmx.de> Tested-by:
-
André Schließer authored
Use default language as fallback for records with sys_language_uid = -1 Resolves: #87067 Releases: master, 9.5 Change-Id: If57d5566b83f4bba79451e8595337ca6c0eb3fc9 Reviewed-on: https://review.typo3.org/59029 Tested-by:
Susanne Moog <susanne.moog@typo3.org> Reviewed-by:
-
Georg Ringer authored
Resolves: #87580 Releases: master Change-Id: I5a384707d0c0ecd9b797ce89dced25f0738942c6 Reviewed-on: https://review.typo3.org/59584 Reviewed-by:
-
Anja Leichsenring authored
The ImageService::getImage requires a string as first and a boolean as third parameter. The ThumbnailViewHelper must respect this requirement. Resolves: #87621 Relates: #87599 Releases: master Change-Id: Ib84f88665fd692629efdb54a33ff83263abe6bfe Reviewed-on: https://review.typo3.org/59617 Reviewed-by:
-
Andreas Fernandez authored
Meta data of files handled by FAL is fetched, created and updated in various places, which makes it hard to maintain the current code base. Albeit the method `_getMetaData()` is marked as internal, it has been marked as deprecated as well, because the method is widely used in the TYPO3 extension universe. For this reason, a MetaDataAspect is introduced that takes care of meta data handling on a low-level basis. In the same run, FAL's `Indexer` is now responsible for creating or updating such meta data records, the `ResourceStorage` now only tells whether auto-extraction is enabled. The meta data extraction, based on registered extractors implementing the `ExtractorInterface` interface, has been moved into a separate service class. Resolves: #85895 Releases: master Change-Id: Icb929a6226777dcea3868ee5c083cf13ff5a71f6 Reviewed-on: https://review.typo3.org/57908 Tested-by:
-
Andreas Fernandez authored
The scheduler now catches any error implementing the `\Throwable` interface thrown by a task to be able to mark such task as executed. Resolves: #87502 Releases: master, 9.5, 8.7 Change-Id: I1f1bebee922c864ce8d7bfc3f0aa6e9434228e98 Reviewed-on: https://review.typo3.org/59504 Tested-by:
-
Stefanos Karasavvidis authored
Use a proper message in case a non existent uid was passed and correctly handle the case where no task uid was passed and no task is overdue. Resolves: #87375 Releases: master, 9.5, 8.7 Change-Id: I69d2798cc185451c889bf37d8a007eacf572c163 Reviewed-on: https://review.typo3.org/59385 Reviewed-by:
Fabien Udriot <fabien.udriot@ecodev.ch> Tested-by:
-
Harald Atteneder authored
substr returns a boolean instead of an empty string. A typecast fixes this problem (See http://php.net/manual/de/function.substr.php#90369) Resolves: #87387 Releases: master, 9.5 Change-Id: I14aa49c18ae028dc27b94bfdbfc3dfb643c18746 Reviewed-on: https://review.typo3.org/59395 Tested-by:
-
Andreas Fernandez authored
The FieldControl `EditPopup` now uses a dedicated module for its handling and no longer uses inline `onclick` handlers nor inline FormEngine calls. Resolves: #86644 Releases: master Change-Id: I46e5d3dbfcb4c187afb6cd1c0692b4e728bfaf26 Reviewed-on: https://review.typo3.org/58887 Tested-by:
Helmut Hummel <typo3@helhum.io> Reviewed-by:
-
Johannes Seipelt authored
- Use strict type mode - Use type hints whereever possible - fix test of creating viewhelper instance using object manager Releases: master Resolves: #87605 Change-Id: Ia626ae1156dacd7c9ddcfefed9bb03d26ba91404 Reviewed-on: https://review.typo3.org/59600 Reviewed-by:
Johannes Seipelt <johannes.seipelt@3m5.de> Reviewed-by:
Felix Döring <Felix.Doering@3m5.de> Reviewed-by:
-
Frank Naegler authored
This patch adds a new ViewHelper to render thumbnails deferred in the backend. This increase the performance of the file list. Resolves: #85607 Related: #85605 Releases: master Change-Id: Id97876e889605d7d8d3075ee98a4fc34b002f395 Reviewed-on: https://review.typo3.org/57646 Tested-by:
-
Josef Glatz authored
Reduce information disclosure of the used TYPO3 version by adding a dynamic 4-digit year in the TYPO3_copyright_year constant. That makes it more difficult to find out the used TYPO3 version just by looking at the frontend source code of a website. Releases: master, 9.5 Resolves: #87543 Change-Id: Ic169548c7111715eecedfce4ced5c8a3564d102a Reviewed-on: https://review.typo3.org/59565 Tested-by:
Jigal van Hemert <jigal.van.hemert@typo3.org> Tested-by:
Stephan Großberndt <stephan.grossberndt@typo3.org> Reviewed-by:
-
Ralf Zimmermann authored
Use the real finisher identifiers as they are configured in the form definition instead the finisher implementation class name. This requires a change of the public AbstractFinisher implementation which implements the FinisherInterface. The interface is not changed, as this would be a breaking change. Therefore, the existence of the new method is checked. If it does not exist in a finisher implementation, a fallback to the previous behavior happens. Releases: master, 9.5 Resolves: #87615 Change-Id: I962502f0601845a5f9793751e87c97b3b902caa1 Reviewed-on: https://review.typo3.org/59606 Tested-by:
-
Gerrit Mohrmann authored
Fixes missing output in ifAuthenticatedViewHelper after cache clearing and erroneous replication of field properties to subsequent fields. composer command used: composer update typo3fluid/fluid Resolves: #86930 Resolves: #86890 Resolves: #87430 Resolves: #86910 Resolves: #87586 Releases: master, 9.5 Change-Id: I8d1abd0aa08d7d7399ec23675dcf07d08dbdd2f2 Reviewed-on: https://review.typo3.org/59554 Tested-by:
Jonas Eberle <flightvision@googlemail.com> Tested-by:
-
Alexander Schnitzler authored
This patch introduces a new Method class to have an easier and more stable api to get information about class methods from ClassSchema objects. The reflection data fetched and stored by/in the ClassSchema objects is no longer publicly available as an array. Instead, Method objects are returned for the following calls: - getMethod - getMethods - getInjectMethods Also it removes the following method: - getConstructorArguments The following constants are marked as private: - ClassSchema::MODELTYPE_ENTITY - ClassSchema::MODELTYPE_VALUEOBJECT Releases: master Resolves: #87427 Change-Id: I38e8a7287896e9303fad1fa8f8f9182895365adc Reviewed-on: https://review.typo3.org/59422 Reviewed-by:
-
