Since the block module is being made optional, we should, for the sake of correctness, add it as a dependency in the .info file,

Comments

andrewlevine’s picture

andrewlevine commented

This is for 7.x, right? As far as I know, there is no precedent for core module dependencies in 6.x info files?

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

Yes, block will be optional in 7.x, but it can still be listed in 6.x. There are other contribs that do list core modules as a dependency. However, even if there weren't that doesn't make it right. There are going to be several upset developers when 7.x comes out and comment and taxonomy are not enabled by default.

intrafusion’s picture

intrafusion
Primary language English
Location Edinburgh, UK
commented
Issue summary: View changes
Status: Active » Fixed
intrafusion’s picture

intrafusion
Primary language English
Location Edinburgh, UK
commented
Version: 6.x-1.x-dev » 7.x-1.x-dev

Status: Fixed » Closed (fixed)

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