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] Readd and fix BE list and order actions #463

Merged
merged 2 commits into from
Apr 2, 2024

Conversation

rintisch
Copy link
Sponsor Collaborator

@rintisch rintisch commented Mar 25, 2024

  • Readds BE action buttons
  • Fix CSV export

Note: I have not tested whether the PDF generation works because there
is no working EXT:cart_pdf for TYPO3 v12.

I do not think that his can be backported due to too many changes in between.

Fixes #421

@rintisch rintisch force-pushed the task/readd-order-actions branch 2 times, most recently from ae76cee to 5daf155 Compare March 26, 2024 12:13
@rintisch rintisch marked this pull request as ready for review March 26, 2024 12:17
The way to add actions in the docheader changed
with https://docs.typo3.org/c/typo3/cms-core/main/en-us/Changelog/11.5/Deprecation-95164-ExtbackendBackendTemplateView.html
which needs adaptions.

Fixes also the CSV export.

This also fixes the issue that numbers could
not be generated because `!isset('')` returns
false.
@rintisch rintisch requested a review from extcode March 26, 2024 12:26
@rintisch rintisch changed the title Readd BE order actions [TASK} Readd and fix BE list and order actions Mar 26, 2024
@rintisch rintisch changed the title [TASK} Readd and fix BE list and order actions [TASK] Readd and fix BE list and order actions Mar 27, 2024
@rintisch
Copy link
Sponsor Collaborator Author

I removed the last commit and will add its result to #465.

Adaptions in `OrderController`:
* Add searchArguments to export,
* Follow CGL
* Remove switch codeblock.
@extcode extcode merged commit 513d973 into extcode:main Apr 2, 2024
8 checks passed
@rintisch rintisch deleted the task/readd-order-actions branch May 22, 2024 06:10
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.

CSV export is broken
2 participants