Skip to content
Snippets Groups Projects
  1. Apr 08, 2024
    • Stefan Bürk's avatar
      [TASK] Add `npm` command dispatcher to `Build/Scripts/runTests.sh` · 20a85842
      Stefan Bürk authored
      `Build/Scripts/runTests.sh` is the monorepo main dispatcher of
      script and tools to ensure same working state across developer
      host system and installation by using well defined containers.
      This mitigates issues with missing dependencies or wrong version
      of tools on develper machines.
      
      Until now local installation of `node` and `npm` in the matching
      version was mandatory to work with the node ecosystem like manage
      dependencies, executing node based tools or the build chain for
      CSS and TypeScript/JavaScript.
      
      This change adds now a `npm` dispatcher to `runTests.sh` similar
      to the `composer` command dispatcher introduced with #103045 to
      allow executing npm commands and defined scripts in `package.json`
      in a well-defined environment and removes the hard-requirement of
      installing it locally for new contributors.
      
      Added `runTests.sh` suite:
      
      * ./Build/Scripts/runTests.sh -s npm --  <npm-command-arguments>
      
      which allows now to using npm commands ...
      20a85842
    • Stefan Bürk's avatar
      [BUGFIX] Ensure working count query in `DatabaseRecordList::getTable()` · 08a78a57
      Stefan Bürk authored
      `\TYPO3\CMS\Backend\RecordList\DatabaseRecordList` provides the ability
      to modify the QueryBuilder used for the record list representation with
      the PSR-14 `ModifyDatabaseQueryForRecordListingEvent`, after generic
      query information like default orderings and similar have been added.
      
      That is covered within the `getQueryBuilder()` method, which is used in
      other class methods, for example in `getTables()` where it is changed
      to a `COUNT()` query.
      
      MariaDB for example is picky about count queries using `order by`,
      `group by` or aggregation methods which are not reflected or added
      in the counter parts leading to a not so useful error like
      
        Mixing of GROUP columns (MIN(),MAX(),COUNT(),...) with no GROUP
        columns is illegal if there is no GROUP BY clause
      
      or similar variants. This error message can be missleading for the
      case a order by has been used along with a COUNT() query.
      
      Listener of the event do not get the information if order-by can
      be added or should be left out, therefore maybe adding sorting
      conditions for a count query.
      
      This change now resets any order statements for the count-query
      like other places in the TYPO3 core.
      
      Resolves: #103427
      Releases: main, 12.4, 11.5
      Change-Id: Ie421993a1890dd6682d02f54203d538e6ae1b76c
      Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83532
      
      
      Reviewed-by: default avatarGarvin Hicking <gh@faktor-e.de>
      Reviewed-by: default avatarStefan Bürk <stefan@buerk.tech>
      Tested-by: default avatarGarvin Hicking <gh@faktor-e.de>
      Tested-by: default avatarcore-ci <typo3@b13.com>
      Tested-by: default avatarStefan Bürk <stefan@buerk.tech>
      08a78a57
    • Georg Ringer's avatar
      [BUGFIX] Use correct icon for pages table · fba8c065
      Georg Ringer authored
      The check for content_from_pid is wrong as it is not the raw database record and therefore the relation is an array.
      
      Resolves: #103405
      Releases: main, 12.4
      Change-Id: I31cfc584d9d1164ef96e1c3bec5ec75cc1619bd7
      Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83662
      
      
      Tested-by: default avatarChristian Kuhn <lolli@schwarzbu.ch>
      Reviewed-by: default avatarChristian Kuhn <lolli@schwarzbu.ch>
      Tested-by: default avatarcore-ci <typo3@b13.com>
      fba8c065
    • Christian Kuhn's avatar
      [BUGFIX] Working link to word details in indexed_search · 7d3c4111
      Christian Kuhn authored
      The indexed_search BE module links to word details.
      This link is broken in main and v12:
      
      * In main, the wordHash argument is a string, but the extbase
        target action expects an int.
      * In v12, the wordHash is an int, but is misinterpreted as
        pages-uid.
      
      The patch fixes this by switching from the argument name
      'id' to 'wordHash' to suppress the pages-uid "convention"
      to kick in. In main, the target action argument is declared
      as string and both arguments are set non-optional.
      
      Resolves: #103547
      Related: #102975
      Related: #96797
      Related: #103540
      Releases: main, 12.4
      Change-Id: Ibd4732d086f1af264ee317c7e7d4fe0f15a15aa4
      Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83680
      
      
      Tested-by: default avatarStefan Bürk <stefan@buerk.tech>
      Reviewed-by: default avatarChristian Kuhn <lolli@schwarzbu.ch>
      Tested-by: default avatarcore-ci <typo3@b13.com>
      Reviewed-by: default avatarStefan Bürk <stefan@buerk.tech>
      Tested-by: default avatarChristian Kuhn <lolli@schwarzbu.ch>
      7d3c4111
  2. Apr 07, 2024
  3. Apr 05, 2024
  4. Apr 04, 2024
  5. Apr 03, 2024
  6. Apr 02, 2024
  7. Apr 01, 2024
  8. Mar 25, 2024
  9. Mar 23, 2024
  10. Mar 22, 2024
  11. Mar 21, 2024
  12. Mar 20, 2024
  13. Mar 19, 2024