{"payload":{"feedbackUrl":"https://github.com/orgs/community/discussions/53140","repo":{"id":1430051,"defaultBranch":"main","name":"typo3","ownerLogin":"TYPO3","currentUserCanPush":false,"isFork":false,"isEmpty":false,"createdAt":"2011-03-02T10:02:23.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/88698?v=4","public":true,"private":false,"isOrgOwned":true},"refInfo":{"name":"","listCacheKey":"v0:1713856269.0","currentOid":""},"activityList":{"items":[{"before":"e4d7215228a8c9378017effebb574890085893b9","after":"f2961dd57a052b4251e4d7ee6357c541e4e37c88","ref":"refs/heads/11.5","pushedAt":"2024-05-10T17:54:58.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[BUGFIX] Prevent TypeError in DatabaseIntegrityController\n\nIn advanced query the operator \"is in list\" makes value\nbe an array. Then switch the field to a non-array field\ntype, and you get the following TypeError:\n\n```\n htmlspecialchars(): Argument #1 ($string) must be of type\n string, array given.\n```\n\nThis resets the array value to an empty string.\n\nResolves: #103487\nReleases: main, 12.4, 11.5\nChange-Id: Ifc6ef7e90d49a05ecb4b6d4869eea13c151b1b2a\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84188\nTested-by: core-ci \nTested-by: Stefan Bürk \nReviewed-by: Stefan Bürk ","shortMessageHtmlLink":"[BUGFIX] Prevent TypeError in DatabaseIntegrityController"}},{"before":"0b06c3e753ff3fae87ef04bbb34669683fe775cb","after":"6fb38b44ae9bed7f72da835d75abcded8bd305de","ref":"refs/heads/main","pushedAt":"2024-05-10T17:54:33.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[BUGFIX] Prevent TypeError in DatabaseIntegrityController\n\nIn advanced query the operator \"is in list\" makes value\nbe an array. Then switch the field to a non-array field\ntype, and you get the following TypeError:\n\n```\n htmlspecialchars(): Argument #1 ($string) must be of type\n string, array given.\n```\n\nThis resets the array value to an empty string.\n\nResolves: #103487\nReleases: main, 12.4, 11.5\nChange-Id: Ifc6ef7e90d49a05ecb4b6d4869eea13c151b1b2a\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83593\nReviewed-by: Stefan Bürk \nReviewed-by: Oliver Klee \nTested-by: Oliver Klee \nTested-by: Benni Mack \nTested-by: Stefan Bürk \nReviewed-by: Benni Mack \nTested-by: core-ci ","shortMessageHtmlLink":"[BUGFIX] Prevent TypeError in DatabaseIntegrityController"}},{"before":"5ef10efb317b1780506245c0e71dd7d6653c2d2a","after":"9612b5939edc63ed9e25e35e99ee2038ef2fa1e0","ref":"refs/heads/12.4","pushedAt":"2024-05-10T17:54:33.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[BUGFIX] Prevent TypeError in DatabaseIntegrityController\n\nIn advanced query the operator \"is in list\" makes value\nbe an array. Then switch the field to a non-array field\ntype, and you get the following TypeError:\n\n```\n htmlspecialchars(): Argument #1 ($string) must be of type\n string, array given.\n```\n\nThis resets the array value to an empty string.\n\nResolves: #103487\nReleases: main, 12.4, 11.5\nChange-Id: Ifc6ef7e90d49a05ecb4b6d4869eea13c151b1b2a\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84198\nTested-by: Stefan Bürk \nReviewed-by: Stefan Bürk \nTested-by: core-ci ","shortMessageHtmlLink":"[BUGFIX] Prevent TypeError in DatabaseIntegrityController"}},{"before":"e8eec80d1ec05486240830044df087aa2cb220e3","after":"5ef10efb317b1780506245c0e71dd7d6653c2d2a","ref":"refs/heads/12.4","pushedAt":"2024-05-10T17:40:25.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[TASK] Add cancel button to dashboard add widget modal\n\nThe add widget modal has a cancel button on the bottom\nof the modal now to match the other modal windows in\nthe backend.\n\nResolves: #103787\nReleases: main, 12.4\nChange-Id: I0c2c970b1148dfddadff1c3722d04dfcc939a69d\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84197\nTested-by: core-ci \nReviewed-by: Stefan Bürk \nTested-by: Stefan Bürk ","shortMessageHtmlLink":"[TASK] Add cancel button to dashboard add widget modal"}},{"before":"420044e055335cd5bd98e22958dbd9d552395288","after":"0b06c3e753ff3fae87ef04bbb34669683fe775cb","ref":"refs/heads/main","pushedAt":"2024-05-10T17:40:07.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[TASK] Add cancel button to dashboard add widget modal\n\nThe add widget modal has a cancel button on the bottom\nof the modal now to match the other modal windows in\nthe backend.\n\nResolves: #103787\nReleases: main, 12.4\nChange-Id: I0c2c970b1148dfddadff1c3722d04dfcc939a69d\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84142\nReviewed-by: Torben Hansen \nReviewed-by: Simon Schaufelberger \nReviewed-by: Georg Ringer \nTested-by: Torben Hansen \nReviewed-by: Stefan Bürk \nTested-by: Simon Schaufelberger \nTested-by: Georg Ringer \nReviewed-by: Jörg Bösche \nTested-by: Stefan Bürk \nTested-by: core-ci ","shortMessageHtmlLink":"[TASK] Add cancel button to dashboard add widget modal"}},{"before":"8b720262353d4ac7ac72fb87566691cc199a6e52","after":"e8eec80d1ec05486240830044df087aa2cb220e3","ref":"refs/heads/12.4","pushedAt":"2024-05-10T17:22:51.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[BUGFIX] Avoid executing anonymous function in Configuration Module\n\nHook implementation are configured with the well known\n\n $GLOBAL['TYPO3_CONF_VARS']['SC_OPTIONS'][...][] = '';\n\nsyntax since ages. Directly defining an anonymous callback function\nis undocumented and rarely used, but technical valid and supported\nby the `GeneralUtility::callUserFunction()` method.\n\nDue to the refactoring of the `ext:lowlevel` Configuration module\nfor the revamped TypoScript parser with TYPO3 v12 the code in the\n`ConfigurationController` has been modified to do an array cast,\nwhich triggers the execution of the function - which can lead to\nsome deep recursions and thus exceeding server resource limits.\n\nDirect callback function should not be executed within the backend\nConfiguration module, which literally is a invalid execution of them.\n\nFollowing code in a extension `ext_localconf.php` file will lead to\nthe resource limit issue:\n\n $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']\n ['tslib/index_ts.php']['preprocessRequest'][]\n = function($parameters, $parent) {};\n\nThis change modifies the `ConfigurationController->renderTree()` to\ncheck if a value is a callable and display the information, and in\ncase of an anonymous function (`Closure::__invoke`) add additional\nstatic information:\n\n Closure::__invoke (anonymous callback function: function() {})\n\nNote: That does not make it obvious where the function is defined,\nbut it avoids calling these callback methods.\n\nResolves: #103813\nRelated: #99899\nReleases: main, 12.4\nChange-Id: I839d1668cf306100cf32af09ec9e24ad0120c43b\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84196\nTested-by: core-ci \nTested-by: Stefan Bürk \nReviewed-by: Stefan Bürk ","shortMessageHtmlLink":"[BUGFIX] Avoid executing anonymous function in Configuration Module"}},{"before":"33a20162b8498ec0ceb3d26168c6a0399b0b5555","after":"420044e055335cd5bd98e22958dbd9d552395288","ref":"refs/heads/main","pushedAt":"2024-05-10T17:22:33.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[BUGFIX] Avoid executing anonymous function in Configuration Module\n\nHook implementation are configured with the well known\n\n $GLOBAL['TYPO3_CONF_VARS']['SC_OPTIONS'][...][] = '';\n\nsyntax since ages. Directly defining an anonymous callback function\nis undocumented and rarely used, but technical valid and supported\nby the `GeneralUtility::callUserFunction()` method.\n\nDue to the refactoring of the `ext:lowlevel` Configuration module\nfor the revamped TypoScript parser with TYPO3 v12 the code in the\n`ConfigurationController` has been modified to do an array cast,\nwhich triggers the execution of the function - which can lead to\nsome deep recursions and thus exceeding server resource limits.\n\nDirect callback function should not be executed within the backend\nConfiguration module, which literally is a invalid execution of them.\n\nFollowing code in a extension `ext_localconf.php` file will lead to\nthe resource limit issue:\n\n $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']\n ['tslib/index_ts.php']['preprocessRequest'][]\n = function($parameters, $parent) {};\n\nThis change modifies the `ConfigurationController->renderTree()` to\ncheck if a value is a callable and display the information, and in\ncase of an anonymous function (`Closure::__invoke`) add additional\nstatic information:\n\n Closure::__invoke (anonymous callback function: function() {})\n\nNote: That does not make it obvious where the function is defined,\nbut it avoids calling these callback methods.\n\nResolves: #103813\nRelated: #99899\nReleases: main, 12.4\nChange-Id: I839d1668cf306100cf32af09ec9e24ad0120c43b\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84187\nReviewed-by: Stefan Froemken \nReviewed-by: Torben Hansen \nTested-by: Torben Hansen \nReviewed-by: Nikita Hovratov \nTested-by: core-ci \nTested-by: Stefan Froemken \nReviewed-by: Stefan Bürk \nTested-by: Nikita Hovratov \nTested-by: Stefan Bürk ","shortMessageHtmlLink":"[BUGFIX] Avoid executing anonymous function in Configuration Module"}},{"before":"e8359a3b8d667fbb74fea77e40722c4f5905405b","after":"33a20162b8498ec0ceb3d26168c6a0399b0b5555","ref":"refs/heads/main","pushedAt":"2024-05-10T13:56:00.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[BUGFIX] Provide configuration with file/folder link result\n\nResolves: #103812\nReleases: main, 12.4\nChange-Id: I467149ad6e93900cf917d3455b24dc01c97330f1\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84186\nReviewed-by: Garvin Hicking \nReviewed-by: Stefan Bürk \nTested-by: Torben Hansen \nReviewed-by: André Buchmann \nTested-by: Garvin Hicking \nTested-by: Stefan Bürk \nTested-by: core-ci \nReviewed-by: Jörg Bösche \nReviewed-by: Torben Hansen ","shortMessageHtmlLink":"[BUGFIX] Provide configuration with file/folder link result"}},{"before":"19a2d873e6974060456080d2340d1c19a49bae26","after":"8b720262353d4ac7ac72fb87566691cc199a6e52","ref":"refs/heads/12.4","pushedAt":"2024-05-10T13:56:00.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[BUGFIX] Provide configuration with file/folder link result\n\nResolves: #103812\nReleases: main, 12.4\nChange-Id: I467149ad6e93900cf917d3455b24dc01c97330f1\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84195\nReviewed-by: Stefan Bürk \nTested-by: core-ci \nTested-by: Stefan Bürk ","shortMessageHtmlLink":"[BUGFIX] Provide configuration with file/folder link result"}},{"before":"96cb08e1aa080361b415c9ec00e9cf04603aa231","after":"e8359a3b8d667fbb74fea77e40722c4f5905405b","ref":"refs/heads/main","pushedAt":"2024-05-10T07:18:57.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[BUGFIX] Allow Resources/Public within asset paths\n\n\"Resources/Public\" may occur also within an\nasset path. Path detection code must therefore\nonly check for its first occurrence.\n\nResolves: #101701\nReleases: main, 12.4\nChange-Id: I8e0b62f5164593e472113ea8d294593e93abfe6c\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84181\nTested-by: Benjamin Franzke \nReviewed-by: Kevin Appelt \nReviewed-by: Benjamin Franzke \nReviewed-by: Garvin Hicking \nTested-by: Garvin Hicking \nTested-by: Josef Glatz \nReviewed-by: Josef Glatz \nTested-by: Willi Wehmeier \nReviewed-by: Willi Wehmeier \nTested-by: core-ci ","shortMessageHtmlLink":"[BUGFIX] Allow Resources/Public within asset paths"}},{"before":"67143a470676668a1a79fff365a14696dadc0e2a","after":"19a2d873e6974060456080d2340d1c19a49bae26","ref":"refs/heads/12.4","pushedAt":"2024-05-10T07:18:40.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[BUGFIX] Allow Resources/Public within asset paths\n\n\"Resources/Public\" may occur also within an\nasset path. Path detection code must therefore\nonly check for its first occurrence.\n\nResolves: #101701\nReleases: main, 12.4\nChange-Id: I8e0b62f5164593e472113ea8d294593e93abfe6c\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84194\nTested-by: Benjamin Franzke \nReviewed-by: Benjamin Franzke \nTested-by: core-ci ","shortMessageHtmlLink":"[BUGFIX] Allow Resources/Public within asset paths"}},{"before":"658dab4fd1ea65783721e84dfefb91cc9e39f055","after":"96cb08e1aa080361b415c9ec00e9cf04603aa231","ref":"refs/heads/main","pushedAt":"2024-05-09T13:53:25.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[TASK] Ensure test function names start with lowercase in ext:form\n\nSome function names for validator tests in ext:form start with an\nuppercase letter, which is not in line with our coding guidelines.\n\nThis change ensures that test function names start with a lowercase\nletter.\n\nResolves: #103809\nReleases: main\nChange-Id: If6d02c44baba9af4c346a6a4652cb929e9f84770\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84184\nTested-by: Oliver Klee \nTested-by: core-ci \nTested-by: Nikita Hovratov \nReviewed-by: Oliver Klee \nReviewed-by: Nikita Hovratov ","shortMessageHtmlLink":"[TASK] Ensure test function names start with lowercase in ext:form"}},{"before":"ce2e70da1c9548e2c5192b9d35eaea60be114c78","after":"67143a470676668a1a79fff365a14696dadc0e2a","ref":"refs/heads/12.4","pushedAt":"2024-05-09T08:43:55.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[DOCS] Add sprintf() reference to f:translate ViewHelper documentation\n\nThe ViewHelper documentation for f:translate mentioned\nsprintf() from PHP, but did not link to it.\n\nThis patch adds a link and also optimizes markup, plus a few more\nsprintf() examples with ordered parameters and number formatting.\n\nSee forge issue for screenshots how this looks before and\nafter the change.\n\nResolves: #103808\nReleases: main, 12.4\nChange-Id: I237d4b668291ad68bf083e35c0bdf6510158b38b\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84193\nTested-by: core-ci \nTested-by: Lina Wolf <112@linawolf.de>\nReviewed-by: Lina Wolf <112@linawolf.de>","shortMessageHtmlLink":"[DOCS] Add sprintf() reference to f:translate ViewHelper documentation"}},{"before":"16b63a1880acd80ad8d0907321f727e61e02aa85","after":"658dab4fd1ea65783721e84dfefb91cc9e39f055","ref":"refs/heads/main","pushedAt":"2024-05-09T08:35:58.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[DOCS] Add sprintf() reference to f:translate ViewHelper documentation\n\nThe ViewHelper documentation for f:translate mentioned\nsprintf() from PHP, but did not link to it.\n\nThis patch adds a link and also optimizes markup, plus a few more\nsprintf() examples with ordered parameters and number formatting.\n\nSee forge issue for screenshots how this looks before and\nafter the change.\n\nResolves: #103808\nReleases: main, 12.4\nChange-Id: I237d4b668291ad68bf083e35c0bdf6510158b38b\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84183\nReviewed-by: Markus Klein \nTested-by: Markus Klein \nTested-by: core-ci \nReviewed-by: Lina Wolf <112@linawolf.de>\nTested-by: Lina Wolf <112@linawolf.de>","shortMessageHtmlLink":"[DOCS] Add sprintf() reference to f:translate ViewHelper documentation"}},{"before":"6199bdd0b0e29e54a1762c786c2344bfe24b0267","after":"16b63a1880acd80ad8d0907321f727e61e02aa85","ref":"refs/heads/main","pushedAt":"2024-05-08T14:22:10.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[TASK] Rename misspelled functional test method\n\nResolves: #103807\nReleases: main, 12.4\nChange-Id: Ifc551dcd66bc88da8c57e7213ee80d960f365509\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84179\nReviewed-by: Andreas Kienast \nTested-by: Benjamin Franzke \nReviewed-by: Benjamin Franzke \nTested-by: core-ci \nTested-by: Andreas Kienast ","shortMessageHtmlLink":"[TASK] Rename misspelled functional test method"}},{"before":"d1309662d936c185e6be64c8d61194fb025d6a01","after":"ce2e70da1c9548e2c5192b9d35eaea60be114c78","ref":"refs/heads/12.4","pushedAt":"2024-05-08T14:22:10.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[TASK] Rename misspelled functional test method\n\nResolves: #103807\nReleases: main, 12.4\nChange-Id: Ifc551dcd66bc88da8c57e7213ee80d960f365509\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84192\nTested-by: Andreas Kienast \nReviewed-by: Andreas Kienast \nTested-by: core-ci ","shortMessageHtmlLink":"[TASK] Rename misspelled functional test method"}},{"before":"9b398933ab6948d8494353cf41fcc38975a2cb8c","after":"6199bdd0b0e29e54a1762c786c2344bfe24b0267","ref":"refs/heads/main","pushedAt":"2024-05-08T08:59:49.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[BUGFIX] Fix positioning of containers in EXT:styleguide BE module\n\nThe containers used in the backend module of EXT:styleguide are now\nbased on flex boxes and don't use absolute positioning anymore. This\nenabled the possibility to scroll when the navigation needs more space\nthan the actual content and also fixes positioning in right-to-left\nscenarios.\n\nResolves: #103794\nReleases: main\nChange-Id: Ie3fcbf7f7141bf16749dc66b45e185168e469260\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84147\nReviewed-by: Andreas Kienast \nTested-by: Frank Nägler \nReviewed-by: Benjamin Franzke \nTested-by: core-ci \nTested-by: Andreas Kienast \nTested-by: Benjamin Franzke \nReviewed-by: Frank Nägler ","shortMessageHtmlLink":"[BUGFIX] Fix positioning of containers in EXT:styleguide BE module"}},{"before":"30399b0eb7d9b08b31f35e1e23cdcfb676ae6aff","after":"9b398933ab6948d8494353cf41fcc38975a2cb8c","ref":"refs/heads/main","pushedAt":"2024-05-07T21:29:08.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[TASK] Avoid usage of ConnectionPool in extbase functional tests\n\nThis change changes extbase persistence tests to use CSV files\nwhere suitable.\n\nThe `addSimpleObjectTest` has been removed, since\n`addObjectSetsDefaultLanguageTest` also tests, if a simple object\ncan be added.\n\nAdditionally, one test is moved from `AddTest.php` to\n`UpdateTest.php`, because this test actually tests an update-\nscenario instead of an add-scenario.\n\nResolves: #103668\nReleases: main\nChange-Id: I4b95a8f6f28be17d50f9f2b88f005a7f02248322\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84153\nTested-by: Benni Mack \nReviewed-by: Oliver Klee \nReviewed-by: Benni Mack \nTested-by: Oliver Klee \nTested-by: core-ci ","shortMessageHtmlLink":"[TASK] Avoid usage of ConnectionPool in extbase functional tests"}},{"before":"1e8e8eb53b77e4d990d46d4dbba9d012a50dfbd5","after":"30399b0eb7d9b08b31f35e1e23cdcfb676ae6aff","ref":"refs/heads/main","pushedAt":"2024-05-07T05:33:08.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[BUGFIX] Allow persisting nullable properties in extbase\n\nExtbase persistence does not treat nullable properties like\nDataHandler does. A nullable domain model property, which\neither implements `DomainObjectInterface` or is of type\n`DateTime`, can not be set to null again, when the object is\nalready persisted with a value not equal to null.\n\nDataHandler respects this scenario by using the value of the\n`default` config TCA key for affected relation fields or by\ntypecasting an empty value to integer for `datetime` fields.\nFor `datetime` fields, additionally the `dbType` is evaluated\nand an empty value is transformed to the corresponding\ndbType empty value.\n\nThis change covers both scenarios in extbase persistence,\nso it is possible to set properties to `null`, if the\nproperty either implements `DomainObjectInterface` or\nis of type `DateTime`.\n\nResolves: #103641\nRelated: #88515\nReleases: main\nChange-Id: I17767b34ac75bb7387a6fc3c6e7345a7d3d25d86\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83839\nTested-by: Simon Schaufelberger \nTested-by: Oliver Klee \nTested-by: core-ci \nReviewed-by: Simon Schaufelberger \nTested-by: Benni Mack \nReviewed-by: Benni Mack ","shortMessageHtmlLink":"[BUGFIX] Allow persisting nullable properties in extbase"}},{"before":"21a80869ff4e46ad72ab9852f826783775c39d98","after":"e4d7215228a8c9378017effebb574890085893b9","ref":"refs/heads/11.5","pushedAt":"2024-05-06T15:41:12.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[TASK] Add unicode license file next to .tbl files\n\nThe folder containing the charset conversion table files\nshould have the unicode license.txt file.\n\nThe license file is from https://unicode.org/license.txt\n\nResolves: #103796\nReleases: main, 12.4, 11.5\nChange-Id: I2577fea6e1b5326cd45da622ec6fbbc563bc140a\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84120\nReviewed-by: Benni Mack \nTested-by: core-ci \nTested-by: Benni Mack ","shortMessageHtmlLink":"[TASK] Add unicode license file next to .tbl files"}},{"before":"f4e04be46e7a5758ffa36a2f7af08b74926393c0","after":"d1309662d936c185e6be64c8d61194fb025d6a01","ref":"refs/heads/12.4","pushedAt":"2024-05-06T15:27:46.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[TASK] Add unicode license file next to .tbl files\n\nThe folder containing the charset conversion table files\nshould have the unicode license.txt file.\n\nThe license file is from https://unicode.org/license.txt\n\nResolves: #103796\nReleases: main, 12.4, 11.5\nChange-Id: I2577fea6e1b5326cd45da622ec6fbbc563bc140a\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84119\nTested-by: Benni Mack \nReviewed-by: Benni Mack \nTested-by: core-ci ","shortMessageHtmlLink":"[TASK] Add unicode license file next to .tbl files"}},{"before":"10882053d043c879348851a932f4296b2ec56ce1","after":"1e8e8eb53b77e4d990d46d4dbba9d012a50dfbd5","ref":"refs/heads/main","pushedAt":"2024-05-06T15:18:30.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[TASK] Add unicode license file next to .tbl files\n\nThe folder containing the charset conversion table files\nshould have the unicode license.txt file.\n\nResolves: #103796\nReleases: main, 12.4, 11.5\nChange-Id: I2577fea6e1b5326cd45da622ec6fbbc563bc140a\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84150\nTested-by: Andreas Kienast \nTested-by: core-ci \nReviewed-by: Benni Mack \nReviewed-by: Andreas Kienast \nTested-by: Benni Mack ","shortMessageHtmlLink":"[TASK] Add unicode license file next to .tbl files"}},{"before":"7c7641ea571ff5ff7a0e56771a70e28e871116cc","after":"f4e04be46e7a5758ffa36a2f7af08b74926393c0","ref":"refs/heads/12.4","pushedAt":"2024-05-06T11:18:58.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[TASK] Fix throws doc comment in MvcPropertyMappingConfigurationService\n\nOne part of the classname in a `throws` doc comment in\n`MvcPropertyMappingConfigurationService` is wrong. Instead of\n`EXTBASE` it should be `Extbase`.\n\nResolves: #103792\nReleases: main, 12.4\nChange-Id: I6be6e53640d82d39ff9f2ce99015877e693b8428\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84118\nTested-by: core-ci \nTested-by: Benni Mack \nReviewed-by: Benni Mack ","shortMessageHtmlLink":"[TASK] Fix throws doc comment in MvcPropertyMappingConfigurationService"}},{"before":"e373ff82b425ae3f367cae575c2ed1e15a9a1fae","after":"10882053d043c879348851a932f4296b2ec56ce1","ref":"refs/heads/main","pushedAt":"2024-05-06T10:06:50.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[TASK] Fix throws doc comment in MvcPropertyMappingConfigurationService\n\nOne part of the classname in a `throws` doc comment in\n`MvcPropertyMappingConfigurationService` is wrong. Instead of\n`EXTBASE` it should be `Extbase`.\n\nResolves: #103792\nReleases: main, 12.4\nChange-Id: I6be6e53640d82d39ff9f2ce99015877e693b8428\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84146\nReviewed-by: Oliver Klee \nTested-by: Oliver Klee \nTested-by: Chris Müller \nTested-by: Benni Mack \nReviewed-by: Benni Mack \nTested-by: core-ci \nReviewed-by: Chris Müller ","shortMessageHtmlLink":"[TASK] Fix throws doc comment in MvcPropertyMappingConfigurationService"}},{"before":"889b486ee93ea444ba237b54f574c7db061827af","after":"7c7641ea571ff5ff7a0e56771a70e28e871116cc","ref":"refs/heads/12.4","pushedAt":"2024-05-05T09:39:10.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[BUGFIX] Update ISO language files\n\nExecuted commands:\n\n composer require --dev sokil/php-isocodes-db-i18n:^4.0.22\n Build/Scripts/updateIsoDatabase.php\n\nResolves: #103791\nReleases: main, 12.4\nChange-Id: I624ce17a6abaa2cb5c0479744db43bdf17cd069a\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84145\nReviewed-by: Andreas Kienast \nTested-by: core-ci \nTested-by: Andreas Kienast ","shortMessageHtmlLink":"[BUGFIX] Update ISO language files"}},{"before":"16bb00d62b0f0b10d32f57ae85f7e3a2cc55fde2","after":"e373ff82b425ae3f367cae575c2ed1e15a9a1fae","ref":"refs/heads/main","pushedAt":"2024-05-05T09:39:10.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[BUGFIX] Update ISO language files\n\nExecuted commands:\n\n composer require --dev sokil/php-isocodes-db-i18n:^4.0.22\n Build/Scripts/updateIsoDatabase.php\n\nResolves: #103791\nReleases: main, 12.4\nChange-Id: I624ce17a6abaa2cb5c0479744db43bdf17cd069a\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84144\nTested-by: Oliver Klee \nTested-by: core-ci \nTested-by: Andreas Kienast \nTested-by: Simon Schaufelberger \nReviewed-by: Simon Schaufelberger \nReviewed-by: Oliver Klee \nReviewed-by: Andreas Kienast ","shortMessageHtmlLink":"[BUGFIX] Update ISO language files"}},{"before":"041497c946cb658ff25d2d42967b5c5b1929a840","after":"16bb00d62b0f0b10d32f57ae85f7e3a2cc55fde2","ref":"refs/heads/main","pushedAt":"2024-05-03T12:28:22.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[TASK] Move table wizard Lit component to correct namespace\n\nThe module `@typo3/backend/element/table-wizard-element` is tightly\ncoupled to FormEngine's `TextTableElement`, therefore said module is\nmoved to the FormEngine namespace. Additionally, the element gets\nrenamed to `typo3-formengine-table-wizard`.\n\nResolves: #103725\nReleases: main, 12.4\nChange-Id: I1c020a4ec6045007ab4c1f42289fe0a9c842f794\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84022\nTested-by: core-ci \nReviewed-by: Benjamin Franzke \nTested-by: Andreas Kienast \nReviewed-by: Frank Nägler \nTested-by: Benjamin Franzke \nTested-by: Frank Nägler \nReviewed-by: Andreas Kienast ","shortMessageHtmlLink":"[TASK] Move table wizard Lit component to correct namespace"}},{"before":"44b7c79e7caa6e778ef101870c7df3cbc1bc1395","after":"889b486ee93ea444ba237b54f574c7db061827af","ref":"refs/heads/12.4","pushedAt":"2024-05-03T12:28:22.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[TASK] Move table wizard Lit component to correct namespace\n\nThe module `@typo3/backend/element/table-wizard-element` is tightly\ncoupled to FormEngine's `TextTableElement`, therefore said module is\nmoved to the FormEngine namespace. Additionally, the element gets\nrenamed to `typo3-formengine-table-wizard`.\n\nResolves: #103725\nReleases: main, 12.4\nChange-Id: I1c020a4ec6045007ab4c1f42289fe0a9c842f794\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84117\nReviewed-by: Andreas Kienast \nTested-by: core-ci \nTested-by: Andreas Kienast ","shortMessageHtmlLink":"[TASK] Move table wizard Lit component to correct namespace"}},{"before":"a512e8bdb61b7623d19e3125289e3fbb9f90870c","after":"041497c946cb658ff25d2d42967b5c5b1929a840","ref":"refs/heads/main","pushedAt":"2024-05-03T09:32:01.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[TASK] Deprecate `MathUtility::convertToPositiveInteger()`\n\nTYPO3 has a method to ensure an integer is always positive:\n`MathUtility::convertToPositiveInteger()`. However, the method is\nrather \"heavy\" as it internally calls `MU::forceIntegerInRange()` and\ntherefore misuses a clamp mechanism to convert the integer to a\npositive number.\n\nAlso, the method name doesn't reflect what it actually does. Negative\nnumbers are not converted to their positive counterpart, but are\nswapped with `0`. Due to the naming issue and the fact that the method\ncan be superseded by a simple `max()` call, the method is therefore\ndeprecated.\n\nResolves: #103785\nReleases: main\nChange-Id: I4fd0548ec6bc7289008057ade4e64f9437c2da7b\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84141\nTested-by: Oliver Klee \nReviewed-by: Torben Hansen \nTested-by: Torben Hansen \nTested-by: core-ci \nReviewed-by: Oliver Klee \nReviewed-by: Benjamin Franzke \nTested-by: Andreas Kienast \nReviewed-by: Andreas Kienast ","shortMessageHtmlLink":"[TASK] Deprecate MathUtility::convertToPositiveInteger()"}},{"before":"a2a017430b290be9cef53604810d7bc6587eff74","after":"a512e8bdb61b7623d19e3125289e3fbb9f90870c","ref":"refs/heads/main","pushedAt":"2024-05-03T07:57:07.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"reviewtypo3org","name":"Replication from review.typo3.org and git.typo3.org","path":"/reviewtypo3org","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5028725?s=80&v=4"},"commit":{"message":"[TASK] Fix error message for PAGEVIEW template\n\nThe $pageLayoutName is treated as 'action' and\nis passed through ucfirst in\nTYPO3Fluid\\Fluid\\View\\TemplatePaths::\nresolveTemplateFileForControllerAndActionAndFormat\n\nTherefore if a backend layout starts with a lowercase name,\nthen the template name still has to start with a uppercase name.\n\nResolves: #103774\nReleases: main\nChange-Id: Ica31c073719d183a5bd66fdaf937dad6707ac07e\nReviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84101\nTested-by: Nikita Hovratov \nTested-by: Benjamin Franzke \nTested-by: core-ci \nReviewed-by: Benjamin Franzke \nReviewed-by: Nikita Hovratov ","shortMessageHtmlLink":"[TASK] Fix error message for PAGEVIEW template"}}],"hasNextPage":true,"hasPreviousPage":false,"activityType":"all","actor":null,"timePeriod":"all","sort":"DESC","perPage":30,"cursor":"djE6ks8AAAAERtBAzgA","startCursor":null,"endCursor":null}},"title":"Activity · TYPO3/typo3"}