in ctools/includes/export.inc, function ctools_export_crud_delete(), line 189

This code doesn't eval the correct db_table_prefix:

 db_query("DELETE FROM {$table} WHERE " . $export['key'] . " = '%s'", $value);

I change the code as

db_query("DELETE FROM {" . $table . "} WHERE " . $export['key'] . " = '%s'", $value);

and it works fine.

Regards Jens

CommentFileSizeAuthor
#6 ctools-872804.patch783 bytespbuyle

Comments

merlinofchaos’s picture

merlinofchaos commented
Status: Active » Fixed

Committed a fix. Thanks for catching this.

Observer123’s picture

Observer123 commented

Hi, I have update but still receive warnings:
* user warning: Table 'payas_drpl1.panels_layout' doesn't exist query: SELECT * FROM panels_layout t__0 WHERE plugin = 'flexible' in /home/payas/public_html/modules/ctools/includes/export.inc on line 379.

* user warning: Table 'payas_drpl1.panels_layout' doesn't exist query: SELECT * FROM panels_layout t__0 WHERE plugin = 'flexible' in /home/payas/public_html/modules/ctools/includes/export.inc on line 379.

merlinofchaos’s picture

merlinofchaos commented

This issue is about a delete query.

If the panels_layout table does not exist, perhaps you need to go hit update.php and run update 6308 again.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

geoman’s picture

geoman commented

Thanks for this. Solved the same problem for me.

pbuyle’s picture

pbuyle commented
StatusFileSize
new ctools-872804.patch783 bytes

We are currently on a tight schedule and unable to update to CTools 1.8 because of unresolved issues with exportable wysiwyg configuration as provided with wysiwyg-624018-with-ui-3.patch in #624018: Exportables and Features support for WYSIWYG 7.x and are affected by this bug. For Drush Make we need to patch to be available somewhere and this issue seems to be a good place (since other may have the same issue and while stuck with CTools 1.7). So here this simple fix as a patch.

truyenle’s picture

truyenle commented

#3 work for me. Thanks a lot.

Truyenle

xgmorkx’s picture

xgmorkx commented

mongolito404: am I correct to pose this is a patch for CTools 1.7 which solves a bug in using the wysiwyg patch in combination with CTools 1.7, but not the main reason why the wysiwyg patch does not work with CTools 1.8? (it might be useful to post a link to this patch in the #624018: Exportables and Features support for WYSIWYG 7.x thread also)

pbuyle’s picture

pbuyle commented

@xgmorkx Yes, the patch I posted in #6 is an patch against CTools 1.7 to solve a bug we experienced when using it with the mentioned wysiwyg patch (but not only). This doesn't solve the incompatibility of CTools 1.8 and this wysiwyg patch.