Grocery Crud Model - set_relation_n_n

[Akshay-Hegde]

Probably we need modification in

function get_relation_n_n_selection_array($primary_key_value, $field_info)
 {

 }

From

$this->db->where($field_info->primary_key_alias_to_this_table, $primary_key_value);

To

$this->db->where('`'.$field_info->relation_table.'`.'.$field_info->primary_key_alias_to_this_table, $primary_key_value);

Column 'group_id' in where clause is ambiguous

SELECT *, CONCAT('', COALESCE(username, ''), '(', COALESCE(email, ''), ')') as s91e87ed5 FROM `default_wf_relation_group` JOIN `default_users` ON `default_wf_relation_group`.`id` = `default_users`.`id` WHERE `group_id` = '3' ORDER BY `default_wf_relation_group`.`priority`

[mckaygerhard]

interesting workaround.. do you tested ¡? @Akshay-Hegde ?

[Akshay-Hegde]

@mckaygerhard Yes, I tested it.

[mckaygerhard]

it's a vbery need workaround i'll tri it in my fork and report some feedback.. very very thanks.. maybe we need a pull request, but i think @scoumbourdis you can integrated without it!

[mckaygerhard]

hi @Akshay-Hegde i tested and i think you have more code changes that are not noted.. i change the line and get similar behaviour ..

your change works only for the filter.. i mean, the "where" part.. but still we have the "colums" part or values/columns that show

so that issue must change the name of the title to "Grocery Crud Model - set_relation_n_n where clause not ambiguos solution" due we still need to clarify wich column will be show if we include the primery key of both tables in relation.. .. i mean still we have missing solution for the columns to show part

[mckaygerhard]

umm @Akshay-Hegde and @scoumbourdis i can see that we can search agains the string "related fiel type" and if we found the common field (well in basic the primary key) we can filter it, remove it and let only one.. what about it?