Grocery_crud_model - get_relation_n_n_selection_array issue

[Akshay-Hegde]

grocery-crud/application/models/Grocery_crud_model.php

Line 340 in c38c119

$results = $this->db->get($field_info->relation_table)->result();

In method

function get_relation_n_n_selection_array($primary_key_value, $field_info)
{

}

Take a example : relation and selection table both are empty, then set relation_n_n something like below

$crud->set_relation_n_n('fields_callback', $this->db->dbprefix('wf_relation_field_callback'), $this->db->dbprefix('wf_field_callback'), 'field_id', 'id', '{field_id}-{name}({remarks})','priority',array());

Will throw you error

{"status":false,"message":"Call to a member function result() on boolean","data":""}

Which is from line 340

$results = $this->db->get($field_info->relation_table)->result();

To fix this issue modify model like below

   	$q = $this->db->get($field_info->relation_table);
	$results = array();
	if($q !== FALSE && $q->num_rows() > 0){
		$results = $q->result();
	}

[Akshay-Hegde]

@mckaygerhard you may be interested, not sure author could test and fix the issue in latest release

[mckaygerhard]

interesting:

$q = $this->db->get($field_info->relation_table);
	$results = array();
	if($q !== FALSE && $q->num_rows() > 0){
		$results = $q->result();
	}

we also need to check if "q" are null object or are not an object ... due the $q->num_rows() check, i thing its a limitation also from the CI framework

[mckaygerhard]

ka ka ka some time since this problem.. now i made a ugly trick due that workaround does not work at all.. the trick consist on create a new table from the customized query.. later render the grocery crud from that recently created table.. of course has a hit performance I/O related.. but works for little projects ...

the "q" must need too many checks.. and workaround related to individuals check..

due lackof time i not implemented a property made solution.. sorry later time i'll try some piece of code