- 2.x branch of http://drupal.org/project/aloha
- Full HTML and Filtered HTML input formats Featurized to replace what's currently in Commons.install.

Members fund testing for the Drupal project. Drupal Association Learn more


lightsurge’s picture

Just adding a reference to the install info http://drupalcode.org/project/aloha.git/blob/refs/heads/7.x-2.x:/README.txt

So as far as 7.x core goes, would currently also require adding a patch to core to establish different filter types (edit: oops, no that's not what this patch does, it does something to drupal_get_js), removing the transformational filters in full/filtered input formats (urls to links / line break conversion), and the 2.x version of jquery_update dev with jquery 1.7 enabled.

erikwebb’s picture

Assigned: Unassigned » erikwebb
erikwebb’s picture

Note: As per https://drupal.org/node/1664602#comment-6339042, this approach does not currently work with JS aggregation enabled.

lightsurge’s picture

Also gives a warning for filters like media token filter that don't tell Aloha what kind of filter they are (transformational etc.). We'll probably keep on getting warned about that until a patch makes it into core which triggers a patch to modules with filters like media.

There's an issue out for core 8.x #807996: [meta] Input filters and text formats

For a start this is only a problem for inline editing which I don't think there's any plans for Commons to use, and if it did I suppose until we have patches we can apply, Aloha will just overwrite tokens with their rendered html. While that's a bit pants it's not exactly the end of the world, so it would seem to make sense to remove this warning from Aloha module in the interim.

erikwebb’s picture

Assigned: erikwebb » Unassigned

Because of the blocking issue, I won't be able to resolve this in a timeframe for my project.

erikwebb’s picture

erikwebb’s picture

Basic patch to -

erikwebb’s picture

Status: Active » Needs review
ezra-g’s picture

Status: Needs review » Fixed

I removed the input format-related code from commons.install and tested this on a fresh install - Seems to be working great. This is committed. Thanks, erikwebb!


Status: Fixed » Closed (fixed)

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