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.

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

Fix for Issue #11 #17

Merged
merged 1 commit into from
Jul 17, 2013
Merged

Fix for Issue #11 #17

merged 1 commit into from
Jul 17, 2013

Conversation

apircher
Copy link
Contributor

Problem: isDirty and reset functions where added to the model. Fixed it
by moving the assignment of "self" out of the result function. So the
this-Reference of the actual dirtyFlag-Object is saved in the variable
self and when the result function is called later, the isDirty and reset
functions are added to the dirtyFlag-Object and not to the viewmodel or the window.

Problem: isDirty and reset functions where added to the model. Fixed it
by moving the self-assignemnt out of the result function. So the
this-Reference of the actual dirtyFlag-Object is saved in the variable
self and when the result function is called later, the isDirty and reset
functions are added to the dirtyFlag-Object.
@dmitry-livchak
Copy link

Dear authors,

Our team has discovered the same issue and I have just submitted a similar pull request.
I am wondering if you would be so kind to process this one?

Kind regards,
Dmitry

hfjallemark pushed a commit that referenced this pull request Jul 17, 2013
@hfjallemark hfjallemark merged commit be71a10 into CodeSeven:master Jul 17, 2013
@hfjallemark
Copy link

Thanks for the pull request @apircher. Sorry for the delay.

This was referenced Jul 17, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants