Problem/Motivation

Follow up to #2686171: WYSIWYG Integration. The settings form allows you to enable the button and the filter format exclusive of each other. Either do this automatically or throw an error to indicate the admins configuration is not complete.

Proposed resolution

As stated above.

Remaining tasks

Patch.

User interface changes

An error on invalid filter configuration.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#8 2687361-error-on-filter-page-8.patch5.64 KBSam152
#2 2687361-error-on-filter-page-2.patch1.8 KBSam152
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Sam152 created an issue. See original summary.

Sam152’s picture

Sam152 CreditAttribution: Sam152 as a volunteer and at PreviousNext commented
Status: Active » Needs review
FileSize
2687361-error-on-filter-page-2.patch1.8 KB

Fixes the issue, but requires a test.

Status: Needs review » Needs work

The last submitted patch, 2: 2687361-error-on-filter-page-2.patch, failed testing.

The last submitted patch, 2: 2687361-error-on-filter-page-2.patch, failed testing.

The last submitted patch, 2: 2687361-error-on-filter-page-2.patch, failed testing.

The last submitted patch, 2: 2687361-error-on-filter-page-2.patch, failed testing.

The last submitted patch, 2: 2687361-error-on-filter-page-2.patch, failed testing.

Sam152’s picture

Sam152 CreditAttribution: Sam152 as a volunteer and at PreviousNext commented
Status: Needs work » Needs review
FileSize
2687361-error-on-filter-page-8.patch5.64 KB

Test fixes + new test for filter config.

  • Sam152 committed f52aa67 on 8.x-1.x 
    Issue #2687361 by Sam152: Make the button and the filter format throw an...
Sam152’s picture

Sam152 CreditAttribution: Sam152 as a volunteer and at PreviousNext commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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