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

Support TYPO3 13 #3

Open
nebrot opened this issue Apr 24, 2024 · 4 comments
Open

Support TYPO3 13 #3

nebrot opened this issue Apr 24, 2024 · 4 comments

Comments

@nebrot
Copy link

nebrot commented Apr 24, 2024

We have tested current branch with TYPO3 13.0.0. With this changes it works for our config:

Change version to to ^12.4 || ^13.0

"typo3/cms-core": "^12.0"

Change version to to 12.4.0-13.4.99

'typo3' => '12.4.0-12.4.99',

Perhaps change deprecated constants (only TYPO3 13?!), see: https://docs.typo3.org/c/typo3/cms-core/main/en-us/Changelog/13.0/Deprecation-101475-IconSizeStringConstants.html#deprecation-101475-icon-size-string-constants

After upgrading to TYPO3 13.1.0 it's not working anymore, perhaps because of new hotkey https://docs.typo3.org/c/typo3/cms-core/main/en-us/Changelog/13.1/Feature-103529-IntroduceHotkeyForSaveAndClose.html#feature-103529-introduce-hotkey-for-save-and-close ?

@gmartino27
Copy link

Also played around and don't found a solution..

Like @nebrot wrote, seems to be related to this:
https://docs.typo3.org/c/typo3/cms-core/main/en-us/Changelog/13.1/Feature-103529-IntroduceHotkeyForSaveAndClose.html#feature-103529-introduce-hotkey-for-save-and-close

Somebody some ideas how we can bring it to work again for TYPO3 ^13.1.0 ??

@svewap
Copy link
Contributor

svewap commented Oct 10, 2024

Sadly, in my current estimation, it is not possible to implement a save-and-close in vendor/typo3/cms-backend/Classes/Controller/EditDocumentController.php, because when the close parameter is set, in line 467 an exit takes place immediately and saving no longer occurs.
I therefore currently only see the change in overloading and modifying the complete class.
What do you think?
I have pushed my status into the release/v13 branch.

// Close document if a request for closing the document has been sent
if ($this->closeDoc > self::DOCUMENT_CLOSE_MODE_DEFAULT) {
    if ($response = $this->closeDocument($this->closeDoc, $request)) {
        return $response;
    }
}

@nebrot
Copy link
Author

nebrot commented Oct 11, 2024

I would prefer to not overloading/modifiying the complete class, seems to be huge and could cause other problems after core-updates.

Do you know which specific core change made it impossible to follow the previous path as in v12? Perhaps we should inform the coredevs that this custom solution doesn't work anymore with current 13.3 and probably 13 LTS next week.

There are many discussions after removing the core solution:

@svewap
Copy link
Contributor

svewap commented Oct 11, 2024

Thanks for the feedback and the references to the discussions. I'm surprised that the hotkey is supposed to work.
https://docs.typo3.org/c/typo3/cms-core/main/en-us/Changelog/13.1/Feature-103529-IntroduceHotkeyForSaveAndClose.html
I'll have to look into it again.

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

No branches or pull requests

3 participants