Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TASK] Migrate javascript modules #3794

Conversation

dkd-friedrich
Copy link
Member

What this pr does

  • Migrate top.fsMod
  • Replace RequireJS for statistics module

How to test

top.fsMod

  1. Open EXT:solr backend module and select a page in the page tree
  2. Select another solr module, check ModuleStateStorage.current('searchbackend')

=> Selected page should be stored in the ModuleStateStorage

SearchStatistics.js

  1. Activate statistics via plugin.tx_solr.statistics = 1
  2. Perform search requests in the frontend
  3. Open statistics in the Solr backend module

=> Statistic chart should be rendered

Fixes: #3489

JavaScript top.fsMod is deprecated since TYPO3 11.4, this commit
replaces the old top.fsMod usage with the ModuleStateStorage,
now the ImmediateActionElement is used.

Resolves: TYPO3-Solr#3489
dkd-friedrich added a commit to dkd-friedrich/ext-solr that referenced this pull request Sep 21, 2023
JavaScript top.fsMod is deprecated since TYPO3 11.4, this commit
replaces the old top.fsMod usage with the ModuleStateStorage,
now the ImmediateActionElement is used.

Ports: TYPO3-Solr#3794
Resolves: TYPO3-Solr#3489
RequireJS integration is deprecated in TYPO3 12 and will be removed
with TYPO3 13, thus search statistics JavaScript is adapted for
ECMAScript v6.

An update to Chart.js 3.9.1 is also done, which provides ES6 support.
Integration in Chart.js 4 is currently faulty and is postponed.

Resolves: TYPO3-Solr#3489
@dkd-friedrich dkd-friedrich force-pushed the task/main/3489-migrate_javascript_modules branch from 91e9306 to 3533d32 Compare September 21, 2023 11:21
Copy link
Collaborator

@dkd-kaehm dkd-kaehm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @dkd-friedrich

@dkd-friedrich dkd-friedrich merged commit fcf82f6 into TYPO3-Solr:main Sep 21, 2023
8 checks passed
@dkd-friedrich dkd-friedrich deleted the task/main/3489-migrate_javascript_modules branch September 21, 2023 13:54
dkd-friedrich added a commit that referenced this pull request Sep 21, 2023
JavaScript top.fsMod is deprecated since TYPO3 11.4, this commit
replaces the old top.fsMod usage with the ModuleStateStorage,
now the ImmediateActionElement is used.

Ports: #3794
Resolves: #3489
@dkd-kaehm dkd-kaehm mentioned this pull request Oct 13, 2023
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[TASK:T12] Migrate top.fsMod.recentIds for BE modules
2 participants