Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna’s picture

Version: 7.x-6.x-dev » 7.x-5.x-dev
Status: Active » Needs review
FileSize
960 bytes

Per the Drupal schema data types documentation, I think that a normal-size blob may be the best option. Need to test it out.

DamienMcKenna’s picture

FileSize
1.58 KB

Updated hook_schema() too.

DamienMcKenna’s picture

Incidentally, this may also fix #1910376: SQL error when importing tweet with emoji.

To be tested.

DamienMcKenna’s picture

Status: Needs review » Closed (duplicate)
DamienMcKenna’s picture

Status: Closed (duplicate) » Needs work

Going to reroll the patches to just add the comments to hook_schema.

DamienMcKenna’s picture

FileSize
1.43 KB

Just the comments.

DamienMcKenna’s picture

Version: 7.x-5.x-dev » 7.x-6.x-dev
FileSize
1.43 KB

Ported to 7.x-6.x.

DamienMcKenna’s picture

Status: Needs work » Needs review
FileSize
1.37 KB

Whoops, shouldn't have removed the 'size' => 'normal' line.

DamienMcKenna’s picture

Version: 7.x-6.x-dev » 7.x-5.x-dev
FileSize
1.37 KB

Ported to 7.x-5.x.

DamienMcKenna’s picture

The last submitted patch, 8: twitter-n2534206-8-7.x-6.x.patch, failed testing.

Status: Needs review » Needs work

The last submitted patch, 10: twitter-n2534206-10-6.x-5.x.patch, failed testing.

DamienMcKenna’s picture

Status: Needs work » Fixed

Committed.

  • DamienMcKenna committed e62bc32 on 7.x-5.x
    Issue #2534206 by DamienMcKenna: More verbose comments in hook_schema to...

  • DamienMcKenna committed 132a9ff on
    Issue #2534206 by DamienMcKenna: More verbose comments in hook_schema to...

Status: Fixed » Closed (fixed)

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