Problem/Motivation

Cascading selection doesn't work.

Proposed resolution

See Patch.

Remaining tasks

Review patch

User interface changes

None

API changes

None

Data model changes

None

CommentFileSizeAuthor
#12 cascading_selection_doesn't_work-2850831.patch1.58 KBhmdnawaz
#6 cascading_selection_doesn't_work-2850831.patch1.15 KBhmdnawaz
term_reference_tree_cascading_problem.patch1.09 KBtomhollevoet

Comments

tomhollevoet created an issue. See original summary.

Status: Needs review » Needs work

The last submitted patch, term_reference_tree_cascading_problem.patch, failed testing.

tomhollevoet’s picture

tomhollevoet commented
Status: Needs work » Needs review

No tests possible? Can somebody check my patch? Thanks!

hmdnawaz’s picture

hmdnawaz commented

The patch is not working in the case when you first select all children and then select the parent. In this case when you then select the parent, the children marked unchecked.

hmdnawaz’s picture

hmdnawaz commented
Status: Needs review » Needs work
hmdnawaz’s picture

hmdnawaz commented
StatusFileSize
new cascading_selection_doesn't_work-2850831.patch1.15 KB

Here is the new patch. Please test it.

hmdnawaz’s picture

hmdnawaz commented
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 6: cascading_selection_doesn't_work-2850831.patch, failed testing.

hmdnawaz’s picture

hmdnawaz commented
Status: Needs work » Needs review
hmdnawaz’s picture

hmdnawaz commented

Please someone test the patch.

hmdnawaz’s picture

hmdnawaz commented
hmdnawaz’s picture

hmdnawaz commented
StatusFileSize
new cascading_selection_doesn't_work-2850831.patch1.58 KB

Status: Needs review » Needs work

The last submitted patch, 12: cascading_selection_doesn't_work-2850831.patch, failed testing.

hmdnawaz’s picture

hmdnawaz commented
Status: Needs work » Needs review
hmdnawaz’s picture

hmdnawaz commented
Priority: Normal » Major
fluidstack’s picture

fluidstack commented

Hi hmdnawaz,

Your patch works. I have heavily tested it with no problems.

alex_optim’s picture

alex_optim
Location Tallinn
commented
Status: Needs review » Postponed
vladimiraus’s picture

vladimiraus
Primary language English
Location Brisbane 🇦🇺
commented
Status: Postponed » Closed (outdated)

Thank you everyone for your contribution.
Drupal 7 is no longer supported but we always need support for Drupal 11 version.
Closing as outdated.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.