Automatic creation of the reciprocal association is not always working as expected. On line 193 that association is made, storing the serialized value of the edited node's NID (which is an integer, not an array). However, uc_upsell_get_associate_nids() expects an array (line 965) and so the automatic reciprocal association is not working as expected as line 965 prevents the association from being returned.

Should be an easy fix, but I'm too new to drupal to start submitting code (sorry!).

Comments

torgospizza’s picture

torgospizza
he/him
Primary language English
Location Portland, OR
commented

Yeah this is the kind of thing that should be fixed with a simple check to see what argument is being received (int or array). I thought I had a fix in for this somewhere. If not I'll resubmit one. Thanks!

jantoine’s picture

jantoine commented
Title: Reciprocal association » Reciprocal association, dissociation
Version: 5.x-2.3 » 6.x-1.x-dev

I believe this is also present in the latest 6.x-1.x-dev release. After associating product A with product B, product B is also associated with product A. When attempting to dissociate product A from product B, product B is still associated with product A. If product B is edited and saved without dissociating product B from product A, product A will once again be associated with product B. Both association and dissociation should be reciprocal.

Cheers,

Antoine