test_dependencies[] can be used to indicate dependencies which are optional but recommended

as per https://drupal.org/node/542202

Let's use this syntax so that users can turn views_ui off on production sites if they so desire.

CommentFileSizeAuthor
#1 panopoly_core-views-ui-independence-1984720-1.patch823 bytesandrew_mallis

Comments

andrew_mallis’s picture

andrew_mallis commented
StatusFileSize
new panopoly_core-views-ui-independence-1984720-1.patch823 bytes

Patch attached.

views_ui should get enabled in panopoly.info instead. Happy to submit another patch if helpful, but it's another one-liner.

pancho’s picture

pancho
Location UTC+2 🇪🇺 EU
commented
Title: views_ui should not be a dependency » Views_ui should not be a hard dependency
Project: Panopoly Core » Panopoly
Version: 7.x-1.0-rc4 » 7.x-1.x-dev
Category: bug » task

I'd rather have panopoly.info introduce a (soft) dependency on Views UI instead of panopoly_core.info's (hard) dependency.
But moving this to Panopoly's main issue queue.

andrew_mallis’s picture

andrew_mallis commented

@Pancho – that'd be my preference too.

populist’s picture

populist commented
Status: Active » Closed (duplicate)

Updated as per #2002084: Remove dependency on Views UI