-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
System Config fields dependency issue at store view level. #767
Comments
Internal ticket: MAGETWO-31535 |
@tzyganu Could you please verify this issue with latest mainline code? We are not able to reproduce the problem. |
@jilu1 If you cannot reproduce it anymore, you can close it. I will take a look in the next few days an reopen it if necessary. |
@jilu1 You are right. This does not reproduce anymore. Thanks for the fix. (I couldn't wait a few more days so I tried it now 😄 ) |
@tzyganu Thank you for your quick response! |
Task - MAGETWO-63236 Add ability to modify simple products data in custom websites
Scenario.
I have 2 fields in
System->Configuration
.The first field is a yes/no (let's call it Enabled).
The second field depends on enabled. Type not important:
I edit the config on the default level and set the value for Enabled to 'Yes'.
I change the the config scope to one of my store views.
Expected Result:
I see the second field "Some Label here" visible and disabled and the checkbox "Use website" checked.
Actual Result
The second field "Some Label here" is visible and enabled and the checkbox "Use website" is checked.
The issue does not reproduce if I remove the dependency (
<depends>
tag) from the second field.I reproduced this while trying to create a config section for one of my modules, but it reproduces also in the core.
Visit
System->Configuration->Customer Configuration->Create New Account Options
.Set
Enable Automatic Assignment to Customer Group
toYes
, Save and change the scope to store view level.Tax Calculation Based On
field (and others) is visible and editable even ifUse Website
is checked.Issue reproduced on alpha 104. I didn't find anything related to this in the alpha 105 change log, so I assumed it still reproduces. I will test on 105 and post what I find.
The text was updated successfully, but these errors were encountered: