- 04 Sep, 2018 10 commits
-
-
Change-Id: I10cf279d74af2cfc2964704ba31c6ce2b7be3493 Reviewed-on: https://review.typo3.org/58180 Reviewed-by:
Oliver Hader <oliver.hader@typo3.org> Tested-by:
-
Resolves: #86126 Releases: master Change-Id: I7724cb1415742861a0c6496f4e3888787e6467d4 Reviewed-on: https://review.typo3.org/58160 Tested-by:
TYPO3com <no-reply@typo3.com> Reviewed-by:
Andreas Fernandez <a.fernandez@scripting-base.de> Reviewed-by:
Susanne Moog <susanne.moog@typo3.org> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
-
In order to correctly work with rootlines (e.g. when resolving according sites and pseudo-sites for URL resolving and generation) possible version move placeholders have to be resolved. Resolves: #86137 Releases: master Change-Id: Ief9841f7592018292c034c03c45343363cf79248 Reviewed-on: https://review.typo3.org/58178 Reviewed-by:
-
The change also contains some documentation related improvements in some code files, where the need occured. Change-Id: I346fd70e20bd0765ac4cc663dc7e3fd73d02651b Resolves: #86075 Releases: master Reviewed-on: https://review.typo3.org/58166 Reviewed-by:
-
When having a pseudo-site the view functionality is broken due to some different handling how to deal with protocols. Resolves: #86134 Releases: master Change-Id: Ia3b08ce968ecfdce631ce9c7ff9c973cfb25eef5 Reviewed-on: https://review.typo3.org/58177 Tested-by:
Wouter Wolters <typo3@wouterwolters.nl> Reviewed-by:
Georg Ringer <georg.ringer@gmail.com> Tested-by:
-
Show the error with a proper markup. Also hightlight type and render type as code. Resolves: #86133 Releases: master, 8.7 Change-Id: I79323aaff834c47e6ff875f8a584b2c88f42c51d Reviewed-on: https://review.typo3.org/58173 Reviewed-by:
Mathias Brodala <mbrodala@pagemachine.de> Tested-by:
-
Due to refactorings lately in BackendUtility::getPagesTSconfig() with which results in a fatal error when clearing page cache while persisting an extbase object. The patch adds a test without a Backend User, and encapsulates this with a simple if() statement. However, this also shows that some other areas that are touched are not fully notice-free / strict, which are tackled as well. Resolves: #85310 Releases: master Change-Id: I0f1318d395451e704e1783e841d96583efe2a560 Reviewed-on: https://review.typo3.org/58174 Reviewed-by:
Daniel Goerz <daniel.goerz@posteo.de> Tested-by:
Markus Klein <markus.klein@typo3.org> Tested-by:
-
If a node in the page tree is clicked, the `nodeSelectedAfter` event is triggered for any active node that is being deactivated. The event listener is now aborted if the event is called on a unchecked node. Resolves: #85787 Releases: master Change-Id: I4fa5bbc4290b5ffd2ecb77a901dceb042dff3d27 Reviewed-on: https://review.typo3.org/57836 Tested-by:
Paweł Rogowicz <rogowicz.pawel@gmail.com> Tested-by:
Tymoteusz Motylewski <t.motylewski@gmail.com> Tested-by:
-
The template module now respects the workspace constraint of the current backend user and the stored sys_template records. Resolves: #86131 Releases: master, 8.7 Change-Id: I0463844c4222957aede4b4ca749a3368cb399f81 Reviewed-on: https://review.typo3.org/58171 Tested-by:
-
Resolves: #86132 Releases: master Change-Id: I34a09ec83491b0f08e0a094680fbd93057ad76a4 Reviewed-on: https://review.typo3.org/58172 Reviewed-by:
-
- 03 Sep, 2018 30 commits
-
-
Fix 1px off error in page tree toolbar. Resolves: #86098 Releases: master Change-Id: Ia1476d57b935159fc32e40a9134c7c7a1e9a6ef3 Reviewed-on: https://review.typo3.org/58167 Reviewed-by:
-
The arguments are now treated as parts of a UTC date/time and prevents different results depending on the system date.timezone settings. Releases: master Resolves: #86128 Change-Id: Ieb336ea43eb246ec0e0c5a7db8194c03a0801f83 Reviewed-on: https://review.typo3.org/58165 Reviewed-by:
-
If two subpages share the same name, it is important, that the first page gets a proper name, and the second one follows the uniqueInSite pattern. Also, sorting by workspaces (live first), PID and then sorting seems to be just logical. I wonder why I did not add this before. Resolves: #86120 Releases: master Change-Id: I53f279034c2c61a31b50fc973721827f0c2685ba Reviewed-on: https://review.typo3.org/58153 Tested-by:
-
This change ensures that workspace version placeholders are kept in sync when slug values (TCA type 'slug') are modified in their specific workspace version. This is done implicitly without the requirement to define shadowColumnsForNewPlaceholders in TCA. Resolves: #86117 Releases: master Change-Id: I11baab1e2154b5f3a043a29c08617c0ad360226a Reviewed-on: https://review.typo3.org/58150 Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
-
BackendWorkspaceRestriction automatically resolves specific records already for a given workspace. In most cases this seems to be fine to omit the usual workspace overlay processing. However, especially when dealing with references and page tree assignments, it just contains wrong information. For instance all workspace versions have the `pid` column set to -1 which of course cannot be resolved as valid page then. Resolves: #85917 Releases: master, 8.7 Change-Id: Ibe20037b25770bea8f58d1a8bba785ca43ac26cb Reviewed-on: https://review.typo3.org/57982 Tested-by:
-
The root page should contain as best practice the "/" as slug, after a successful installation. Resolves: #86125 Releases: master Change-Id: I2dced238f3482564f4c23fae6c191b68fc500b8e Reviewed-on: https://review.typo3.org/58162 Reviewed-by:
-
Resolves: #86127 Releases: master Change-Id: I2b900305cb505c52886b012d7d9a3303f6a0a233 Reviewed-on: https://review.typo3.org/58163 Reviewed-by:
-
Resolves: #86124 Releases: master Change-Id: Icb706e585af8e0c0acd993f15d17276024bc5503 Reviewed-on: https://review.typo3.org/58157 Reviewed-by:
Anja Leichsenring <aleichsenring@ab-softlab.de> Tested-by:
-
Resolves: #86118 Related: #85937 Releases: master Change-Id: Id913e98e360d06f8db61bbde8c49a131715f2240 Reviewed-on: https://review.typo3.org/58151 Reviewed-by:
Richard Haeser <richard@maxserv.com> Reviewed-by:
-
If a page title is like "!!!" this currently results in an invalid slug. The generator now auto-generates something with "default-hashXYZ" in order to always be unique on creation time. Resolves: #86122 Releases: master Change-Id: I453e2155c29f1bde4d05d481e0ae43913bc5ad8b Reviewed-on: https://review.typo3.org/58155 Reviewed-by:
-
On database systems like sqlite fields like uid or t3ver_oid are returned as string and need to be casted. This casting has been added in PreviewUriBuilder ->getLivePageUid. Resolves: #86119 Releases: master Change-Id: I7fec760b0aee749e72a640488e1021681471c9ec Reviewed-on: https://review.typo3.org/58152 Reviewed-by:
-
Because the usage of $GLOBALS['TSFE']->altPageTitle is deprecated since version 9 and will be removed in version 10, the whole class can be deprecated. Resolves: #86121 Releases: master Change-Id: Ifc846550d1132ac15e35fcd7e0357a45c015ee53 Reviewed-on: https://review.typo3.org/58154 Reviewed-by:
Björn Jacob <bjoern.jacob@tritum.de> Tested-by:
-
Some small indentation fixes. Resolves: #86123 Releases: master Change-Id: Idd5cc7f9cb206516930e5a888449d1204ee9ae11 Reviewed-on: https://review.typo3.org/58156 Reviewed-by:
-
When upgrading with e.g. MSSQL it should be noted, that the pages.slug DB field should be a varchar database field (instead of "text"), which can be null, allowing the field to be created when upgrading an instance, without having the upgrade wizard run yet. So all slug fields are set to null, and can be ommitted correctly. Resolves: #86116 Releases: master Change-Id: Iedf62d1a145c25d0ecbc9e25c163255173cdb2d7 Reviewed-on: https://review.typo3.org/58146 Tested-by:
-
This reverts commit 277c677a. This change was meant as a pre-cursor to the new config handling, which needs some more thoughts on how to deal with AdditionalConfiguration.php in the future, and is reverted for the time being. Resolves: #86115 Reverts: #84623 Releases: master Change-Id: Ib46666856b3ed797f574f10f2af024d29bdad956 Reviewed-on: https://review.typo3.org/58147 Reviewed-by:
Jigal van Hemert <jigal.van.hemert@typo3.org> Tested-by:
-
Using a TCA type=user with a custom renderType is much more elegant since the registered element can then fully access $this->data and has control of node expansion and all the other goodies that come with own renderType in comparison to the shabby limited type=user with userFunc. The patch switches the 'is_public' type=user userFunc to a proper renderType and fixes the incomplete HTML of this field along the way. Note a type=check with own renderType is not possible in this case since the config has no items array. Resolves: #86109 Releases: master Change-Id: I72507a4660fc64e5c2df60768564a788a4c38a05 Reviewed-on: https://review.typo3.org/58141 Tested-by:
-
TranslationService of forms only respect language sets in TypoScript. Use SiteLanguage is the leading configuration and TypoScript as fallback. Resolves: #85909 Releases: master Change-Id: I961161434e38916e8885af883523eff98f0b467c Reviewed-on: https://review.typo3.org/57969 Tested-by:
Julian Wundrak <julian@wundrak.net> Tested-by:
-
When NOT using FormEngine to create a new page (e.g. in the pagetree or via the "Create multiple pages" wizard), so just using DataHandler magic is uttermost critical that a slug is always created, to ensure the "uniqueInSite" functionality, and to access a page, even though the page was created after the upgrade wizard has run. Resolves: #86050 Resolves: #85937 Releases: master Change-Id: I9eff3385c369a04a6f5a33d0b840b6a2b698891c Reviewed-on: https://review.typo3.org/58098 Tested-by:
-
The database table `sys_lockedrecords` should only be cleared via the BackendUtility when a Backend user is logging off. Currently, this is also called for Frontend Users, which actually removes everything from the currently logged-in backend users with the same uid. As this is very bad code design on many levels (lockRecords for unlocking, no context for the authentication user object etc), this should be encapsulated within the BackendUser object directly anyway. For further abstractions, this could also be a hook or something else, to be even cleaner. Resolves: #86113 Releases: master, 8.7 Change-Id: I44d91064edb6ec9ef4c148e48b67bdf22da38869 Reviewed-on: https://review.typo3.org/58145 Reviewed-by:
-
When invoking slug generation on an empty or base slug (`/`), the slug will become `//` - which is invalid and has to be avoided. Resolves: #86114 Releases: master Change-Id: I66aac94ec9be7992376806d59b7c3ef1bd0e142a Reviewed-on: https://review.typo3.org/58144 Tested-by:
-
The patch introducing interfaces for upgrade wizards had some mixed namings, which are now corrected. Resolves: #86112 Releases: master Change-Id: I5ada026d00f9482b1c89fed63cb40a9ba6fb790e Reviewed-on: https://review.typo3.org/58143 Reviewed-by:
-
The newly added functionality for typolink.language within TypoScript receives a documentation file. Resolves: #86057 Releases: master Change-Id: I87031125f879950f926e118bf4b4bcc5a068f39f Reviewed-on: https://review.typo3.org/58142 Reviewed-by:
-
When using a Site according "slug" values have to be given in order to have complete routing information. There won't be pages in sites that explicitly have empty slugs. Test cases for sites having slugs are available in SlugLinkGeneratorTest. In order to test legacy behavior without having sites and slugs, the scenario of LinkGeneratorTest has been adjusted accordingly. Resolves: #86104 Releases: master Change-Id: Iea57e62311fc663df772c12530a0c4d4c84aee0f Reviewed-on: https://review.typo3.org/58140 Reviewed-by:
-
Resolves: #85549 Releases: master Change-Id: I8a98da8842c6d15175debed5513bbc4bc2bb0a2c Reviewed-on: https://review.typo3.org/57586 Tested-by:
-
Also add an example how to pass in multiple language ISO codes which was done using commas back then and now using spaces. Resolves: #85694 Related: #82315 Releases: master Change-Id: I4088b6d313c90e5d882db243ea134250690808cf Reviewed-on: https://review.typo3.org/58139 Tested-by:
Guido Schmechel <guido.schmechel@brandung.de> Reviewed-by:
-
Susanne Moog authored
The upgrade wizard API has been in need of some cleanups both architectural as well as functional. As a result the API is now easier to use and more streamlined, and commands available can also be listed. Resolves: #86106 Resolves: #86101 Releases: master Change-Id: I1b5eb3459f0650ac1ed9e85718bfa34eb07e403d Reviewed-on: https://review.typo3.org/58128 Reviewed-by:
-
Due to the refactoring process within links to a site, it is still necessary to link to a translated page with a given "&L=" parameter. This parameter is resolved, and a link to a page translation is generated. Fixes menu generation and typolink with the generated page title. Resolves: #86067 Releases: master Change-Id: I3e1208a2cdb438c68d4ed3dac1d0274ce07395dc Reviewed-on: https://review.typo3.org/58108 Reviewed-by:
-
This change was not necessary since multiple values can be passed in separated by spaces instead of commas just like any other Symfony command. Example: bin/typo3 language:update de ja This reverts commit f612f2e5. Change-Id: Ie87d8245082613b4185b859f635bf712094ca914 Resolves: #85694 Releases: master Reviewed-on: https://review.typo3.org/58137 Tested-by: -
As long as there is no site configuration record to edit, the title displayed into Sites Module should not carry a link. Change-Id: I3c177d4ba889a224577ab53c4d2350e04d57049e Resolves: #86105 Relates: #86069 Releases: master Reviewed-on: https://review.typo3.org/58138 Reviewed-by:
-
Now it is possible to pass several Isocodes comma separated by CLI and to update them. Releases: master Resolves: #85694 Change-Id: I1a618d6ed8c3abda650995d5d7d1eb523cff1bc1 Reviewed-on: https://review.typo3.org/57786 Tested-by:
-
