I'd like to clean up our /includes folder a bit, and this is the common pattern.

CommentFileSizeAuthor
#1 1703468.flag_.move-views-includes.patch83.48 KBjoachim

Comments

joachim’s picture

joachim commented
Status: Active » Needs review
StatusFileSize
new 1703468.flag_.move-views-includes.patch83.48 KB

Patch is on top of mine at #1699752: split flag.inc into different classes.

Testbot will probably complain :)

Status: Needs review » Needs work

The last submitted patch, 1703468.flag_.move-views-includes.patch, failed testing.

socketwench’s picture

socketwench commented
Status: Needs work » Needs review

#1: 1703468.flag_.move-views-includes.patch queued for re-testing.

socketwench’s picture

socketwench commented

Re-testing as #1699752: split flag.inc into different classes is committed.

socketwench’s picture

socketwench commented
Status: Needs review » Fixed

Tested, committed, marking as fixed.

joachim’s picture

joachim commented

Oops. hook_views_api needs to be changed too.

Committed this patch.

Issue #1703468, fixup, by joachim: Fixed broken path in hook_views_api().

Status: Fixed » Closed (fixed)

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