-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
Not working with TYPO3 v11.5.24+ and news v11 #15
Comments
Same here. |
Thank you for adding support of TYPO3 11 and news 10 last year. Is it very complex here to add support for news 11? |
Hey guys, I've created a PR for TYPO3 11 compatibility. Could any one of you please have a look and see if that branch would be working for you? Thaaaaanks ;-) |
Hi, I followed the changes as in the webpage: 378f1e9. |
Update: |
@Tonyt8, you're running TYPO3 12, right? This issue here is about TYPO3 11... but OK, let's create compatibility with 12 too! ObjectManager is gone in TYPO3 12, so we need to replace that bit... |
@Tonyt8, please have another go with the latest version of my branch! |
@kitzberger Thanks for making it compatible with version 12.4, cool! right now it's not working for me. I made the latest changes to the files and installed. What I have done so far: Today I applied these changes combined with yesterday's changes. Migrate from ObjectManager to Dependency Injection. To: NewsListActionEventListener.php, EnrichDemandObject.php and Configuration/Services.yaml. In EXT I have the directory: news_filter installed with extension manager. Only for testing I added the form html to the list.html file. Located: /ext/news/Resources/Private/Templates/News/List.html I have double checked the all changes I made to files. |
Perhaps the Log error might help: | Core: Exception handler (WEB): Uncaught TYPO3 Exception: The file "-hosting.de/httpdocs/typo3cms/f-a/typo3conf/ext/news_filter/Configuration//Services.yaml" does not contain valid YAML: Duplicate key "public" detected at line 11 (near "public: true"). \| Symfony\Component\DependencyInjection\Exception\InvalidArgumentException thrown in file hosting.de/httpdocs/typo3cms/typo3_src-12.4.2/vendor/symfony/dependency-injection/Loader/YamlFileLoader.php in line 780. Requested URL: veen.nl/typo3/module/tools/ExtensionmanagerExtensionmanager/Action/installExtensionWithoutSystemDependencyCheck?token=--AnonymizedToken--&extensionKey=news_filter -- | -- |
@Tonyt8, can you pls show me your version of the Services.yaml? |
Hi, services: GeorgRinger\NewsFilter\Hooks\EnrichDemandObject:
|
@Tonyt8, what about the indention of those lines? Sorry, but I can't really tell from your pasting. Anyway, it needs to be indented properly. |
@kitzberger |
@kitzberger Before starting to intergrate the News_filter I tested only using the standard news extension without any additions. Step1. Setup the news extension and the page plugins in the standard way and made some news item with categories. The news item worked as expected. My intension is to first get the normal news extension working correctly before testing the news_filter again. Do you have any idea what might be causing the above 404 issue and what I could try to solve it? Many thanks. Regards, |
🤔 Since you're not working with composer, why don't you just do a git clone of my fork's branch? So you don't have to manipulate files manually? cd typo3conf/ext
rm -rf news_filter
git clone --branch typo3-11 [email protected]:kitzberger/news_filter.git |
I'm testing your (@kitzberger) branch I noticed that the FlexForm setting (That said, IMO it doesn't make sense to load a page in the flexform setting |
@cepheiVV, for both installations you're using EXT:news 11.3.0, I assume? Is Can you please check? |
However, when I create a sys_category records on that assigned page, I get an error in the frontend.
In
|
It's fixed on branch |
Thanks for the fix! That works on v12! @kitzberger |
Package ist not installable using TYPO3 v11.5.24+ and news v11
The text was updated successfully, but these errors were encountered: