ezpublish-legacy icon indicating copy to clipboard operation
ezpublish-legacy copied to clipboard

EZP-27299: search index not updated if obje…

Open bernhard-roessler opened this issue 8 years ago • 5 comments

Ticket: https://jira.ez.no/browse/EZP-27299 Search index is not updated if a object is hidden by the cronjob (hide.php).

bernhard-roessler avatar Apr 25 '17 12:04 bernhard-roessler

Any reason why the standard hide operation would not work in this scenario? Maybe there is a bug somewhere to be fixed instead of adding this workaround ?

gggeek avatar Apr 25 '17 12:04 gggeek

What do you mean with "standard hide operation"? I assume the "eZContentObjectTreeNode::hideSubTree( $node );" is already a standard operation. In our cases it did not register the object for reindexing. Maybe there is a better place to put this change, but for us it was the "nearest" one :-)

siegfried-zach avatar Apr 25 '17 12:04 siegfried-zach

maybe @gggeek is talking about this one? https://github.com/ezsystems/ezpublish-legacy/blob/master/kernel/classes/ezcontentobjecttreenode.php#L5983 Looking there https://github.com/ezsystems/ezpublish-legacy/blob/master/kernel/classes/ezcontentobjecttreenode.php#L6020 i don't know if that op does a reindex of the content, but if not, maybe the code you added can be moved here...

c-revillo avatar Apr 25 '17 12:04 c-revillo

And this is the problem with (missing) layers in legacy.. For the most part use of eZSearch is done outside of persistence classes, like for instance in operations classes: https://github.com/ezsystems/ezpublish-legacy/blob/master/kernel/content/ezcontentoperationcollection.php#L1011

andrerom avatar Apr 29 '17 07:04 andrerom

I agree. The search index update should happen in https://github.com/ezsystems/ezpublish-legacy/blob/master/kernel/content/ezcontentoperationcollection.php.

Bernhard, would the function changeHideStatus work for you?

pkamps avatar May 17 '17 08:05 pkamps