-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
CHE-4939: Fix bugs related to processing of read-only files #5003
Conversation
ResourceDelta delta = event.getDelta(); | ||
Path resourceLocation = delta.getResource().getLocation(); | ||
|
||
if (project.equals(resourceLocation) && REMOVED == delta.getKind()) { |
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.
Check for delta kind at first, then for equality of resources. It will be faster than this condition.
|
||
private void removeHandlers() { | ||
if (fileEventHandlerRegistration != null) { | ||
fileEventHandlerRegistration.removeHandler(); |
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.
Set fileEventHandlerRegistration
to null
.
} | ||
|
||
if (resourceChangeHandlerRegistration != null) { | ||
resourceChangeHandlerRegistration.removeHandler(); |
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.
Set resourceChangeHandlerRegistration
to null
also.
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.
Other ok
Build success. https://ci.codenvycorp.com/job/che-pullrequests-build/2543/ |
Build success. https://ci.codenvycorp.com/job/che-pullrequests-build/2547/ |
Signed-off-by: Roman Nikitenko <rnikitenko@codenvy.com>
Build # 2604 - FAILED Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/2604/ to view the results. |
Signed-off-by: Roman Nikitenko rnikitenko@codenvy.com
What does this PR do?
Fix bugs related to processing of read-only files:
What issues does this PR fix or reference?
#4939
Changelog
Fix bugs related to processing of read-only files