-
Notifications
You must be signed in to change notification settings - Fork 19
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
Preview target rebuild in engine instead of http call #395
Preview target rebuild in engine instead of http call #395
Conversation
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.
Please also add to the Deployer Upgrade Manager (UM) to automatically remove it from existing installations.
@sumerjabri I added the removal processor to UM. It will match for the same processor with the same properties for removal. |
* @return true if all properties are the same, false otherwise | ||
*/ | ||
protected boolean hasAllProperties(Map<String, Object> processorObj) { | ||
if (processorObj.get(PROCESSOR_NAME_CONFIG_KEY).equals(processorName)) { |
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.
This condition can be inverted to reduce indentation
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.
I updated this. Thank you!
import static org.craftercms.deployer.impl.DeploymentConstants.PROCESSOR_NAME_CONFIG_KEY; | ||
|
||
/** | ||
* Operation to remove a new processor to the pipeline of the target |
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.
"Operation to remove a processor from the pipeline of the target"
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.
I updated this. Thank you!
Ticket reference or full description of what's in the PR
craftercms/craftercms#6223