ezsystems/ezfind — pull request #142 by guillaumelecerf (2013-11-28 16:37:28+00:00)
Title: Fix EZP-16372: fix the check for ezfind.ini\[IndexOptions]\DisableDelete...

...Commits

The prototypes of removeObject() and removeObjectById() differ between eZSearch and eZFind, i.e:

static function removeObject( $contentObject, $commit = true )
versus
function removeObject( $contentObject, $commit = null )

When eZContentObject calls eZSearch::removeObjectById( $delID ), removeObjectById() is called by eZSearch with its default value, $commit = true.

So the check :
if ( !isset( $commit ) && ( $this->FindINI->variable( 'IndexOptions', 'DisableDeleteCommits' ) === 'true' ) )

is always false, because $commit is always defined.

So simply remove this check, and let ezfind.ini\[IndexOptions]\DisableDeleteCommits determine if we have to commit or not.