Hi Toopay,

I had already made the change to core.php, I tried using the core.php you linked to in case there were other changes but it still doesn’t work. 

With the code I posted you can see that the all() method works one way along the relationship with the patch, but not the other.  I put the behaviour that’s failing also at the bottom of the post, where with

$token points towards the correct values in $holder

However $original_id points towards the primary key value (a site_code value), and not the foreign key value (a manager_refid value) which would produce an array result.

Thanks for the help

snowfall - 06 November 2012 07:12 AM

However $original_id points towards the primary key value (a site_code value), and not the foreign key value (a manager_refid value) which would produce an array result.

Gotcha. I’ve just update the previous patch for core.php : https://gist.github.com/4023670

@toopay

Thanks for the reply.  The issue with the category model is that the relationships are in a separate table, so I can’t query the category table directly.  Each category can have multiple parents, and multiple children

@toopay

And if there’s no way to modify the query on the foreign table through the pivot table, how would you recommend I execute a more complex query on it?  I guess I’ll have to bypass the built-in relationship functionality.

Thanks

Is there a way to chain multiple where() clauses together?  I’ve tried it, but only the last clause is in the query.

When i updated the core, this error started showing:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ‘“event_images” WHERE “event_images”.“events_id” IN (1, 2, 3, 4, 5, 6, 7, 8, 9)’ at line 1

SELECT * FROM “event_images” WHERE “event_images”.“events_id” IN (1, 2, 3, 4, 5, 6, 7, 8, 9)

My guess is that the double quotes carries the problem!

Updated patch : https://gist.github.com/4023670 (i’ve added backward compability within `generate_clause` method, as @necken issue emerged).

Soon i have enough time i will wrap those patch with appropriate unit-test files, and merge that into the main repo. Report any issue with this patch, if any.

@jonahdahlquist,

And if there’s no way to modify the query on the foreign table through the pivot table, how would you recommend I execute a more complex query on it

Only ‘select’, ‘limit’ and ‘order_by’ clauses which were supported, right now. That were more than enough for me

Regarding ‘where’ clause, ‘where’ method is accept three type of parameters. First you could passed a string :

Then you could also passed an array. So theres no need to chain ‘where’ with other ‘where’, since it will overwrite current recorder with the last one. But you COULD, chaining ‘where’ with ‘or_where’. For further detail, check this test case

@Toopay,

Thanks for the patch, it works perfectly <edit> I had a brief look but could not see in the source just now, when passing the $POST data to create a record does gasorm escape the data, or perform the XSS check?  If not, is there a point in the code you would recommend to add it, so it performs on all data supplied for insert/update and where fields. </edit>

Hi Toopay, another bug I think.

After applying the patches I have this error:

In the Where clause after the period it wrongly puts the table name connected to the primary key, it uses ‘tbl_sample_sample_refid’ when it should only be ‘sample_refid’.

The relationships appear set up properly as are the primary/foreign key values in both models.

Calling Line:

Also same error with ->get(1) instead of ->all().

Tbl_sample exert:

Tbl_sample_related_context exert:

Hi,

started using Gas ORM today and am very pleased with it, but am having trouble with something. I don’t know how to do multiple join statements. Searched everywhere but was not able to find solution.

Thanks

@Kljofa

Hello,

What do you mean when you say multiple joins?  Do you mean one parent entity with many child entities, or a parent entity with a child entity, which in turn is the parent of another and so on.  For the first type of relationship the documentation gives good examples. First you need to set up the relationships so that gasorm knows how tables relate to each other:

http://gasorm-doc.taufanaditya.com/relationship.html

Once you have done that, you can use the with() function.  Here is an example from:

http://gasorm-doc.taufanaditya.com/quickstart.html

There is an entity called user, and the relationships between the (has many)kid, (has one)wife and (has many)job entities have been created so that the command below creates the joins and retrieves the corresponding records for all users.

If there is a chain of parent/child entities, the above command could be used to get the initial parent/child relationship, then the command below can be used to fetch only the child entity details of further entities.

Depending on the source of your download you may have to watch your capitalization as the link below discusses, as I vaguelly remember I had the problem with capitalization too first time round:

http://ellislab.com/forums/viewthread/213348/P120/#1010854

@snowfall,

Its looks like the ORM class seeing your primary key as invalid collumn name, in the constructor.

Just to verify, ensure that ‘sample_context_refid’ is exists on below script output :

@toopay

Result is:

The code nearly works, it is using the correct foreign key except it has the table name attached.  This is the malformed SQL:

This is what it *should* be when correct:

@snowfall.

I can’t replicate this behaviour, since custom foreign key actually already well-covered within recent unit-test suite.

I suspect, the ORM class going down to this line while investigating your entity meta-data.

Here is a crude example of my problem.

I have 3 tables

TABLE 1
id
year

TABLE 2
id
table1_id

TABLE 3
id
table2_id

I want to grab all results from table 3 that matches year from table 1.

I tried doing something like this:

but i only get one join and that’s the last one i entered. Relationships are properly set.