zikula-modules / Clip

Flexible Document Creator
35 stars 8 forks source link

Upgrade from PageMaster 0.4.2 fails - see PR #135 #160

Closed rmaiwald closed 10 years ago

rmaiwald commented 10 years ago

This is PR #135 from TkdAndi (2012-12-28), which has been closed without code changes.

I got similar error messages like TkdAndi while upgrading from PageMaster 0.4.2 to Clip 0.9.3 under Zikula 1.3.7. After testing PR #135 and doing some additional changes the upgrade worked and all looks fine.

What does this patch do? All new Clip database tables have to be accessed through Doctrine, only the old PageMaster database tables should be accessed with DBUtil::getLimitedTablename() or DBUtil::metaColumnNames(). The two DBUtil functions add the old prefix to the table name, but the new Clip tables do not have a prefix anymore. The additional function getLimitedTableNameWithoutPrefix() defined by TkdAndi is not neccessary.

matheo commented 10 years ago

Thanks for this! :)