Odd behaviour with Active Record

Hey guys,

for your information i'm using CI 3:

Can anyone of you reproduce what i've experienced;
I've a compound select query like:

PHP Code:

$query = $this->db

->select("ur.*, ca.name, GROUP_CONCAT(ug.id SEPARATOR ',') AS usergrouplist",false)... 


Now, my problem is the generated Query

Code:

SELECT ur.*, ca.name, GROUP_CONCAT(ug.id SEPARATOR ', ') AS usergrouplist

There is a space after the comma
Am i overlooking something or is there a bug in CI ?

For too advanced queries I can suggest you to input normal query inline..

Code:

$sql = "SELECT * FROM some_table WHERE id = ? AND status = ? AND author = ?";
$this->db->query($sql, array(3, 'live', 'Rick'));

Check here http://www.codeigniter.com/user_guide/da...eries.html

Best VPS Hosting : Digital Ocean

Yeah you are right, i can do that, but i'm to lazy to write full queries
On the other hand, is this suggestion a recommendation or just a workaround for my problem because in my case the whole query looks like

PHP Code:

$query = $this->db
    ->select("ur.*, ca.name, GROUP_CONCAT(ug.id SEPARATOR ',') AS usergrouplist",false)
    ->from("UserRights AS ur")
    ->join("Categories AS ca","ca.id = ur.Item_id","inner")
    ->join("UserGroups_UserRights AS uu", "uu.UserRights_id = ur.id","inner")
    ->join("UserGroups AS ug","ug.id = uu.UserGroups_id","inner")
    ->where("ca.Modules_id", $module_id)
    ->where("ur.typ","Category")
    ->where("ur.name","read")
    ->where("ca.active",1)
    ->group_by("ur.id")
    ->order_by("ca.name")
    ->get(); 


And i really don't want to set aside the benefits of the Query Builder...

Yeah you are right, i can do that, but i'm to lazy to write full queries
On the other hand, is this suggestion a recommendation or just a workaround for my problem because in my case the whole query looks like


And i really don't want to set aside the benefits of the Query Builder...

You know that Query Builder is just a tool to generate correct query. So at the end you always get 1 line query..

At least from my experience if you have Big and Complicated query, its harder to use a Query Builder than to write it plain. 
So in some cases the better choice is a plain query..

In other cases where you have dynamic dependences QB is the best choice.

Best VPS Hosting : Digital Ocean

https://github.com/bcit-ci/CodeIgniter/issues/634

@sintakonte

For this particular case you may try a temporary workaround about the bug for now. Use $this->db->get_compiled_select() method and see the generated $sql. Then, by using string search and replace on the $sql correct ', ' to ','. And then use $this->db->query($sql).

This problem results out of a core function inside the

Code:

DB_query_builder.php

locatet under /system/core/database

Here you can search for the protected function

Code:

protected function _compile_select

on line 2259 (on CI 3.x).

Code:

$sql .= implode(', ', $this->qb_select);

So here must be changed the space for the implode function. But - correct me if iam wrong - its not possible to extend the Database Classes without a deeper hack.

thx for your answers guys

PHP Code:

$query = $this->db
    ->select(array("ur.*, ca.name, GROUP_CONCAT(ug.id SEPARATOR ',') AS usergrouplist"),false)... 


i tested the array solution and it works like a charm