lochmueller / calendarize

📆 Best TYPO3 Calendar ever 📆
http://typo3.org/extensions/repository/view/calendarize
75 stars 84 forks source link

TYPO3 v12 support #734

Closed zobnov closed 4 months ago

zobnov commented 1 year ago

Any plans to update the cms-core version so that the extension could be installed with composer on Typo3 12?

Below is my current output with error for reference: —— Problem 1

lochmueller commented 1 year ago

Yes... but I think it is not directly on the agenda in the next weeks, because EXT:autoloader and EXT:calendarize must be compatible. I will check this in the next weeks and also check, if this could be part of some integration projects...

zobnov commented 1 year ago

@lochmueller thank you! Any suggestions on how could I install it today? I guess I could try changing the version in the composer file somewhere manually. Or maybe I could install the development version?

lochmueller commented 1 year ago

Hey @zobnov there is no dev version that is compatible. You can fork the repository, change the composer.json to fit qour requirements and install from your repository (just add your repo to your compoaser.json, you do not need change the package name). If you fix things, you could do this directly in your repository. If the changes are good, we cann merge this changes via pull requests back to the main repo. Regards, Tim

zobnov commented 1 year ago

Sounds good, I’ll try this out and will let you know. Thank you.

okmiim commented 1 year ago

A list of things to be check and migrated (probably incomplete):

Future when dropping v11 support

lochmueller commented 1 year ago

Hey @okmiim Hey @zobnov

Sebastian Fischer is currently prepare a v12 compatible version. Please check the forks to get insides of his work. Hey also decouple the extension from EXT:autoloader. He will work a few days and will send a pull request.

I suggest to wait for his work and check, if there are open points after the check/merge.

Regards, Tim

lochmueller commented 1 year ago

@garbast FYI

sandrotanner commented 1 year ago

A lot of works has been going on in the mentioned forks ( https://github.com/garbast/calendarize and https://github.com/garbast/calendarize_pages ), thanks a lot @garbast :pray:. The have been inactive for about a month now though so I wanted to ask: Do they need more work? Or is it practically ready and we can help with testing? Can we do something to help?

garbast commented 1 year ago

Hi. Sorry for the stale development, but i was quite busy with the project, where i use calendarize in.

In general the work is done. There are some things (the backend modul) that need some love, but in general i'm quite happy with the state.

sandrotanner commented 1 year ago

Cool, no worries. Thank you!

kreiseuskirchen commented 1 year ago

You think a release is soon to come?

romyschmidt commented 9 months ago

How is the status now?

lochmueller commented 9 months ago

Hey @romyschmidt

Still the same. Currently, the v12 branch is not mergeable https://github.com/lochmueller/calendarize/pull/755 if the branch is mergeable and we get some feedback, we will merge this and finish the backend module. Feel free to check, why the PR could not merged and send some Feedback if there are any problems with this fork.

Regards, Tim

okmiim commented 7 months ago

Known tasks/problems regarding TYPO3 v12

And as always: more testing

hannesbochmann commented 7 months ago

Maybe the flexform migration can be done with EXT:mktools. I added recently a command to do the migration of plugins with switchable controller actions in a generic way. @see https://github.com/DMKEBUSINESSGMBH/typo3-mktools/blob/12.4/Documentation/Utilities/Extbase/Index.md

hannesbochmann commented 7 months ago

Here are some example commands for the migration:

bin/typo3 mktools:migrate-switchable-controller-actions --actions="Calendar->detail" --list-type=calendarize_calendar --new-list-type=calendarize_detail

bin/typo3 mktools:migrate-switchable-controller-actions --actions="Calendar->list" --list-type=calendarize_calendar --new-list-type=calendarize_list

This worked for me pretty well.

hannesbochmann commented 7 months ago

@okmiim Can you tell me what exactly doesn't work anymore in the old method in the AbstractCompatibilityController when it comes to the feed rendering? As far as I can see the main/only problem is the retrieval of the response body. Right now "$response->getBody()->getContents()" is used. As the response got written previously the stream pointer is at the end of the stream and getContents() will start to output from where the pointer is which is the end so nothing will be echoed. The correct method to use should be "$response->getBody()->__toString()" which rewinds the stream beforehand. I can provide a MR with the changes if there is no other problem.

hannesbochmann commented 7 months ago

Besides that I just installed calendarize in a 12.4 project with a fairly complex setup and custom events. So far I didn't run into any problems besides those which are already fixed/known.

okmiim commented 7 months ago

@hannesbochmann

Feeds: your are right with the feed contents - thank you

Regarding switchable controller actions: I would prefer to have a migration integrated into the extensions, since it has a better user experience and everybody needs to do this. Not everyone is such tech savvy as you.

hannesbochmann commented 7 months ago

Yeah, I completely understand that. I try to find some time to add the migration. Would you prefer a command or a update wizard?

lochmueller commented 7 months ago

Yeah, I completely understand that. I try to find some time to add the migration. Would you prefer a command or a update wizard?

Thank you! I would prefer an update wizard, because we use this already in the past: https://github.com/lochmueller/calendarize/tree/master/Classes/Updates

lochmueller commented 5 months ago

Hey @okmiim , what do you think about a v12 release in the next days?! My plan is...

lochmueller commented 4 months ago

I close this issue (thanks for all the work. The release is already there ;) )... there are two followups:

Feel free to create more issues for open stuff.