TYPO3-Solr / ext-solr

A TYPO3 extension that integrates the Apache Solr search server with TYPO3 CMS. dkd Internet Service GmbH is developing the extension. Community contributions are welcome. See CONTRIBUTING.md for details.
GNU General Public License v3.0
136 stars 246 forks source link

[BUG] Solr configuration not found because the root page is not configured #3407

Closed ochorocho closed 1 year ago

ochorocho commented 1 year ago

Describe the bug

When editing e.g. a news record on a page where solr is not configured

Call to a member function getSolrConfiguration() on null

To Reproduce

Expected behavior

Do not update or index items on a page that is not configured to use solr.

Screenshots grafik

Used versions:

Additional context Add any other context about the problem here.

stigfaerch commented 1 year ago

I also experience the same thing when using the DataHandler in my own code to create content. The patch in the pullrequest (github.com/ochorocho/ext-solr/pull/1) from @ochorocho fixed the problem for me.

dkd-kaehm commented 1 year ago

@ochorocho The pull request must go against this repository not on your fork.

dkd-kaehm commented 1 year ago

@ochorocho On the way, please add "Fixes: #3407" in the commits message body as well.

ochorocho commented 1 year ago

@dkd-kaehm 🤦 sorry for the mess. Created the PR against this repository, added "Fixes" to the commit message.