-
Notifications
You must be signed in to change notification settings - Fork 133
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
Renaming a method results in loss of code #143
Comments
Thanks for reporting the issue ! In fact it is a normal behavior: when renaming a method from a schema and saving, here what we do:
So to be able to keep the behavior, we need to be sure that we were in the case of a renaming and not of a deleting / adding. Need to check how to do that. |
Hi Erwan,
I understand the issue you're facing.
May I suggest a diff type popup to give the user the chance of specifying
what to do?
In the meantime, perhaps you could mention this behaviour in the readme?
Tjerk
…On Wed., Jan. 5, 2022, 9:17 a.m. Erwan Carriou, ***@***.***> wrote:
Thanks for reporting the issue !
In fact it is a normal behavior: when renaming a method from a schema and
saving, here what we do:
1. we compare the new schema with the old one,
2. we notice that the method name is not more there, so we remove it
from the model and remove its behavior (if any)
3. we also notice that a new method name is there, so we add it to the
model
So to be able to keep the behavior, we need to be sure that we were in the
case of a renaming and not of a deleting / adding. Need to check how to do
that.
—
Reply to this email directly, view it on GitHub
<#143 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ALMMFGZC6DLU6RV3GICXYNTUURHIBANCNFSM5LI3WALQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
The documentation has been updated to inform users about this behavior (see https://designfirst.io/systemdesigner/documentation/docs/en/create-a-schema.html#define-a-method) . |
Awesome 😀
…On Fri., Jan. 7, 2022, 5:48 p.m. Erwan Carriou, ***@***.***> wrote:
The documentation has been updated to inform users about this behavior
(see
https://designfirst.io/systemdesigner/documentation/docs/en/create-a-schema.html#define-a-method)
.
—
Reply to this email directly, view it on GitHub
<#143 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ALMMFG3QO6J43GAV34A2ZZ3UU5UU7ANCNFSM5LI3WALQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Describe the bug
When you rename a method in Schema the definition in Model is completely replaced and implementation of the method disappears completely.
When the method is renamed back to its original name, the definition in Model is again completely erased and the implementation of that method is reset to the default template - the original code is gone.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
I expect the name of the method to be updated in Model and in the implementation of the method and I expect the code of the implementation to remain untouched.
Desktop (please complete the following information):
The text was updated successfully, but these errors were encountered: