Without this, the module receives a red X where one would normally enable it, and a message "This version is incompatible with the 6.0-dev version of Drupal core."

CommentFileSizeAuthor
#3 Picture 4_5.png44.67 KBwebchick

Comments

douggreen’s picture

douggreen commented
Category: bug » support

There already is a check for this. See coder_6x.inc line 460. And I just re-tested it on the 5.x filter module, and it indeed does display an error:

Line 10: New syntax for .info files requires core=6.x
webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Active » Closed (fixed)

D'oh! You are right. I must've been tired last night. :)

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Title: 6.x upgrade: Warn of missing core = 6.x in .info files. » .info files should have their own file entry
Category: support » task
Status: Closed (fixed) » Active
StatusFileSize
new Picture 4_5.png44.67 KB

Well... let's approach this differently then, as this is why I thought there wasn't a warning for this.

Right now, the .info file warning is in under views.module's entry. If it was broken up into its own entry, it would be easy to tell what needed to change in the .info file specifically.

See screenshot.

nancydru’s picture

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

Minor technicality that I see in the picture, "...has been completely re-hauled..." should be "...has been completely over-hauled..."

douggreen’s picture

douggreen commented
Status: Active » Postponed

Putting the info file in it's own fieldset is a little difficult because they are handled by custom code in the review include. I'm going to postpone this for later.

However, I just fixed the line sorting of errors. This used to work, but I think was broken 2-3 months ago with other changes. By fixing the line sorting, these .info files always show up at the top of the list and are easier to see.

@nancyw, I also fixed the wording.

douggreen’s picture

douggreen commented
Status: Postponed » Closed (fixed)

Housekeeping