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

[FEATURE] Update extension for TYPO3 v12 support #35

Merged
merged 2 commits into from
Feb 8, 2024
Merged

Conversation

calien666
Copy link
Member

@calien666 calien666 commented Jan 14, 2024

With this PR the extension is shippable for TYPO3 v12. This adds support
for the backend integration.

This PR is ready for testing

@calien666 calien666 force-pushed the prepare-v12 branch 14 times, most recently from d72375a to dca1f9d Compare January 14, 2024 21:04
The update adds a new controller, DataSheetImportController, for
handling data import from excel sheets. This includes supporting files,
templates and necessary configurations. Also, there are enhancements to
user access check functions with better considerations for user
permissions. A testing framework is also being introduced though actual
tests are not yet added.

This is preparation work for updating this extension for v12/v13 support
@calien666 calien666 marked this pull request as ready for review January 14, 2024 21:06
@calien666 calien666 changed the title [WIP][FEATURE] Update extension for TYPO3 v12 support [FEATURE] Update extension for TYPO3 v12 support Jan 14, 2024
@calien666 calien666 force-pushed the prepare-v12 branch 3 times, most recently from 9d16021 to 3757aee Compare January 14, 2024 21:40
Copy link

@sbuerk sbuerk left a comment

Choose a reason for hiding this comment

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

TBH only did a pure code review only, no testing in running instance for any versions.

Nice move, but some minor remarks from my side. 😇

ext_tables.php Outdated Show resolved Hide resolved
Classes/Service/ImportService.php Outdated Show resolved Hide resolved
Classes/Service/ImportService.php Outdated Show resolved Hide resolved
}

/** @deprecated Function call will be removed in 6.0 */
$insertData = $this->callHook($insertData, $importJob->getTable());
Copy link

Choose a reason for hiding this comment

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

See comment in the callHook() method. I would suggest to consider dropping the hook directly.

Classes/Service/ImportService.php Outdated Show resolved Hide resolved
Classes/Service/ImportService.php Outdated Show resolved Hide resolved
@ErHaWeb
Copy link
Contributor

ErHaWeb commented Jan 15, 2024

Successfully tested the following functions in a running v12.4.9 instance:

  • Import of tt_address data based on an xlsx-File
  • Access to the "XLS Importer" module by non-admin users with appropriate access

With this PR the extension is shippable for TYPO3 v12. This adds support
for the backend integration.
@calien666 calien666 merged commit 4a42894 into master Feb 8, 2024
8 of 12 checks passed
@calien666 calien666 deleted the prepare-v12 branch February 8, 2024 20:11
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.

3 participants