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

Status field is dependent on state field but it not responding to changes #323

Closed
yunusyerli1 opened this issue Aug 18, 2023 · 2 comments · May be fixed by #400
Closed

Status field is dependent on state field but it not responding to changes #323

yunusyerli1 opened this issue Aug 18, 2023 · 2 comments · May be fixed by #400
Labels
Area: Frontend Priority:Important Issues & PRs that are important; broken functions; errors; there are workarounds Status: Fix Proposed A issue that has a PR related to it that provides a possible resolution Status:Requires Updates Issues & PRs which requires input or update from the author Type: Bug Something isn't working

Comments

@yunusyerli1
Copy link
Contributor

yunusyerli1 commented Aug 18, 2023

Issue

he status field in the Cases module is showing all values no matter what value is selected in the State field.

Expected Behavior

“New” and “In progress” Status drop-down values will be displayed when the State field is set to “Open”.
“Resolved” and “Cancelled” Status field drop-down values will be displayed when the State field is set to “Closed”.

Actual Behavior

The status field in the Cases module is showing all values no matter what value is selected in the State field.

Possible Fix

checkAndInitAsDynamicEnum() method need to be added to dropdownenum.component. Because the method checks the values depending on a dependant field.

Steps to Reproduce

  1. In base-field.manifest.ts file change following:
    From :
    'enum.detail': EnumDetailFieldComponent,
    'enum.edit': EnumEditFieldComponent,
    To:
    'enum.detail': DropdownEnumDetailFieldComponent,
    'enum.edit': DropdownEnumEditFieldComponent,
  2. In editview, select State and check Status field. You ll see all values.
    3.Select another value in State, and still you will see all the values

Context

Status needs to be dynamic dependent on State field. This is a low priority.

Your Environment

  • SuiteCRM Version used: 8.3.0
  • Browser name and version (e.g. Chrome Version 51.0.2704.63 (64-bit)): Chrome Version 115.0.5790.171 (64-bit)
  • Environment name and version (e.g. MySQL, PHP 7): Wamp Server, PHP 7.4.33, MariaDB 10.4.12
  • Operating System and version (e.g Ubuntu 16.04): Windows 11
yunusyerli1 added a commit to yunusyerli1/SuiteCRM-Core that referenced this issue Aug 18, 2023
@johnM2401 johnM2401 added Priority:Important Issues & PRs that are important; broken functions; errors; there are workarounds Status:Requires Updates Issues & PRs which requires input or update from the author Status: Fix Proposed A issue that has a PR related to it that provides a possible resolution Area: Frontend labels Aug 31, 2023
@johnM2401
Copy link

johnM2401 commented Aug 31, 2023

Hey Yunus!

I've given this a look and it appears to be Dynamically updating, as expected

See gif:

caseDyn

Is there anything further that is required to replicate this?
(ie: Changing the displayType to dropdown for these fields? or something else?)

Or have I perhaps missed something?

Thanks!

@johnM2401 johnM2401 added the Type: Bug Something isn't working label Aug 31, 2023
yunusyerli1 added a commit to yunusyerli1/SuiteCRM-Core that referenced this issue Oct 5, 2023
yunusyerli1 added a commit to yunusyerli1/SuiteCRM-Core that referenced this issue Nov 7, 2023
yunusyerli1 added a commit to yunusyerli1/SuiteCRM-Core that referenced this issue Nov 7, 2023
mpuyosa91 pushed a commit to yunusyerli1/SuiteCRM-Core that referenced this issue Nov 9, 2023
mpuyosa91 pushed a commit to mpuyosa91/SuiteCRM-Core that referenced this issue Nov 15, 2023
mpuyosa91 pushed a commit to yunusyerli1/SuiteCRM-Core that referenced this issue Nov 15, 2023
mpuyosa91 pushed a commit to mpuyosa91/SuiteCRM-Core that referenced this issue Dec 27, 2023
@lukio lukio mentioned this issue Feb 2, 2024
6 tasks
AlexMyddleware added a commit to AlexMyddleware/SuiteCRM-Core-fix-323 that referenced this issue Feb 28, 2024
@johnM2401
Copy link

Hey @yunusyerli1
Thanks for raising this issue!

As of the most recent release, (8.6.0), the issues with Dynamic Dropdowns should now be resolved.

See gif, showing the behaviour:

dynamicDD

I will close this ticket off for now, but please get back in touch if you are experiencing further issues with this.
Thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Frontend Priority:Important Issues & PRs that are important; broken functions; errors; there are workarounds Status: Fix Proposed A issue that has a PR related to it that provides a possible resolution Status:Requires Updates Issues & PRs which requires input or update from the author Type: Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants