Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fixes MER#1039 #151

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fixes MER#1039 #151

wants to merge 1 commit into from

Conversation

lpotter
Copy link
Contributor

@lpotter lpotter commented Jun 1, 2015

Make sure to update services properties

@lpotter
Copy link
Contributor Author

lpotter commented Jun 1, 2015

Originally, it did not pass the cached properties, but change 5f73bcc added this aspect.
The signal emiter will only emit signals for the properties that have actually been changed.

@net147
Copy link
Contributor

net147 commented Jun 1, 2015

setPath calls resetProperties which will remove each property from the properties cache and emit changed signals for each property removed.

@lpotter
Fixes MER#1039
1b79479 
Make sure to update services properties
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lpotter @net147