As marked by a TODO comment, the 

DataHandler->truncate()

method should be converted to d7 API.

This is a simple conversion, so find attached a simple patch that does this.

CommentFileSizeAuthor
#4 data-truncate-api-2762713-4.patch458 bytesm1n0
#2 data-truncate-api-2762713-2.patch475 bytesm1n0

Comments

m1n0 created an issue. See original summary.

m1n0’s picture

m1n0 commented
Status: Active » Needs review
StatusFileSize
new data-truncate-api-2762713-2.patch475 bytes
joachim’s picture

joachim commented
Status: Needs review » Needs work

Thanks for the patch!

Though it looks like escaping the table name is taken care of by core: https://api.drupal.org/api/drupal/includes%21database%21query.inc/functi...

m1n0’s picture

m1n0 commented
StatusFileSize
new data-truncate-api-2762713-4.patch458 bytes

Ah of course, I missed that, here is the rerolled patch.

m1n0’s picture

m1n0 commented
Status: Needs work » Needs review
jerry’s picture

jerry commented
Status: Needs review » Reviewed & tested by the community

Working fine here. Let's RTBC it.

  • joachim committed 04fbb47 on 7.x-1.x authored by m1n0 
    Issue #2762713 by m1n0: Converted truncate() method to use Drupal 7...
joachim’s picture

joachim commented
Status: Reviewed & tested by the community » Fixed

Committed. Thanks everyone!

Status: Fixed » Closed (fixed)

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