i activated ec_recurring module, tables were built in database with no problems

the problem is when i try to access myfiles section, when i get this warning:

user warning: Table 'reloadmagazine_db.ec_recurring_schedule' doesn't exist query: SELECT * FROM ec_recurring_schedule ORDER BY name in /Users/theclue/Sites/reloadcms/includes/database.mysqli.inc on line 151.

the point is that all my tables are prefixed with 'reload_': ec_recurring_* tables don't exist in my database schema...

another bug (probably linked): when i try to add a schedule schema i get a success status message but no schema is created...

Comments

The.Clue’s picture

The.Clue commented

digging a bit, i checked ec_recurring.module by hand...all table names SEEMS to be correctly aliased....

sounds weird...

The.Clue’s picture

The.Clue commented
Status: Active » Fixed

found it...it was in row 670...

replace with:

$table = ec_recurring_get_table('{ec_recurring_schedule}', 'ec_recurring_rowFn_schedules', 'name');
The.Clue’s picture

The.Clue commented
Title: ec_recurring doesn't consider table prefix » a better fix

sorry for my latest post. my fix works, but it's not the best solution.

this is probably more elegant:

1. don't touch row 670

2. replace row 1936 with:

$result = db_query(empty($qry_override) ? "SELECT * FROM {" . $tablename . "}$orderby" : "$qry_override$orderby");

i just altered this part

FROM {$tablename}$orderby

and now it works...dunno why...but it works....

neclimdul’s picture

neclimdul
He/Him
commented
Title: a better fix » ec_recurring doesn't consider table prefix
neclimdul’s picture

neclimdul
He/Him
commented
Status: Fixed » Needs review

Wait... I don't think this has been committed so it isn't fixed...

gordon’s picture

gordon commented
Status: Needs review » Fixed

I just checked my system and it has been fixed.

baraban’s picture

baraban commented
Status: Fixed » Active

It is not fixed in the release I downloaded today.
Wasted at least an hour fighting this one...
Hoepfully we will soon be able to use it on the Odesskiy Listok newspaper website at http://www.OdessaPage.com

neclimdul’s picture

neclimdul
He/Him
commented
Status: Active » Fixed

I actually thought this was fixed too. In the future actual patches help clarify your goals even simple one liners.

The problem btw has to do with php string parsing. {$variable} is an explicit variable declaration so you could have "{$variable}string" and php knows "string" isn't part of the variable name. That's the php lesson for today. ;)

Committed the fix. I'm sure more little bugs will come in and we'll get another release soon enough. Until then, patches for 5.x-3.0
and 4.7.x-3.0

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)