Pull Request for Issue # .

Summary of Changes

In Smart Search you can disable entries in the index by unpublishing them. Unfortunately, the entries were re-published when an item was saved again. I earlier claimed that it wasn't possible to prevent that behavior in our current system because I thought the state of the content item and the state of the item in the index were the same field in the database. I was wrong and we actually have a field named "published" for the state of the item in the index and "state" for the state item of the content item. Sorry, @brianteeman 😉 (Maybe you can help me find the issue where this was reported? I can't find it right now...)

After some debugging it turned out that in our finder plugins we are using the reindex() method of the Adapter class in finder and that method falsely completely deletes the entry first from the index and then adds it back in again. This actually is far less performant than simply indexing again as the normal indexer does...

Testing Instructions

Index your content and unpublish an entry in the index. Edit the content and without changing anything, save it again.

Actual result BEFORE applying this Pull Request

The item in the index is published again.

Expected result AFTER applying this Pull Request

The item stays unpublished until you publish it again.

Link to documentations

Please select:

• Documentation link for docs.joomla.org:

• No documentation changes for docs.joomla.org needed

• Pull Request link for manual.joomla.org:

• No documentation changes for manual.joomla.org needed