-
Notifications
You must be signed in to change notification settings - Fork 242
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
Update search index in hide cronjob #1348
Conversation
@@ -49,6 +49,9 @@ | |||
{ | |||
$cli->output( 'Hiding node: "' . $node->attribute( 'name' ) . '" (' . $node->attribute( 'node_id' ) . ')' ); | |||
eZContentObjectTreeNode::hideSubTree( $node ); | |||
|
|||
//call appropriate method from search engine |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nitpick: // Call appropriate ..
Seems legit. I wonder if there are other cases in the legacy kernel where we miss similar calls... eg: content/copy.php first creates a copy and only after it updates the visibility => if copy an hidden node, does the copy gets indexed as visible or invisible ? |
@l0rdJ Seems some new commits have been introduced here now, possible to cleanup tp focus on the |
@andrerom I pointed source branch for this PR. |
@l0rdJ Ok, but this PR is good to go now right? |
Yep, it safe to merge it to master as it is |
@@ -90,6 +90,7 @@ eZOEPopupUtils.settings.onInitDoneArray.push( function( editorElement ) | |||
ezoeLinkAttribute.timeOut = setTimeout( eZOEPopupUtils.BIND( ezoeLinkAttribute.ajaxCheck, this, url[0] + '_' + id, lid ), 320 ); | |||
return true; | |||
}); | |||
inp.keyup(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@l0rdJ I think you accidentally merge in unrelated changed here, or?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, merged that first so it got resolved when merging this, thanks @l0rdJ !
* Update search index in hide cronjob (cherry picked from commit 6f4e99f)
* Fix link source change (object => node) when editing a version * Update search index in hide cronjob
https://jira.ez.no/browse/EZP-27299