Welcome Guest, Not a member yet? Register   Sign In
DB Active Record MAX, MIN, etc. fix - I think...
#3

[eluser]gregmax17[/eluser]
Oops, I found an error in my script already...

In the DB driver class, where Rick's comment is, I suggest to remove that IF statement or comment the return. But in fact, it should be this:

Code:
if (strpos($item, '(') !== FALSE && strpos($item, '.') === FALSE)
{
    return $item.$alias;
}

Basically I am checking if we are going to add a table prefix to the item or not.

Again, I am just testing this on PyroCMS, but I encourage everyone to use it to bullet proof this method.

@phil
I finally forked your awesome PyroCMS and using these changes, got it to install the table prefix's without any errors


Messages In This Thread
DB Active Record MAX, MIN, etc. fix - I think... - by El Forum - 03-11-2010, 12:05 AM
DB Active Record MAX, MIN, etc. fix - I think... - by El Forum - 03-17-2010, 09:52 AM
DB Active Record MAX, MIN, etc. fix - I think... - by El Forum - 03-17-2010, 11:14 AM



Theme © iAndrew 2016 - Forum software by © MyBB