Hi,

files[] directive should only be used for files containing classes / interfaces for the ClassLoader. Let's remove the others!

CommentFileSizeAuthor
#2 field_group-1911530.patch492 bytestobiasb
#1 field_group-1911530-1-useless_info_files.patch468 bytesSimon Georges
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Simon Georges’s picture

Simon Georges CreditAttribution: Simon Georges commented
Status: Active » Needs review
FileSize
field_group-1911530-1-useless_info_files.patch468 bytes

And the patch.

This patch is part of the #1day1patch initiative.

tobiasb’s picture

tobiasb
Primary language German
Location Berlin
CreditAttribution: tobiasb commented
Issue summary: View changes
FileSize
field_group-1911530.patch492 bytes

Updated the patch for the current dev release.

nils.destoop’s picture

nils.destoop CreditAttribution: nils.destoop commented
Status: Needs review » Fixed

Thx, code has been committed to dev.

Status: Fixed » Closed (fixed)

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

  • Commit 90c8bed on 7.x-1.x, 8.x-1.x authored by Simon Georges, committed by zuuperman: 
    Issue #1911530 by tobiasb, Simon Georges: Remove useless files[]...

Status: Closed (fixed) » Needs work

The last submitted patch, 2: field_group-1911530.patch, failed testing.

nils.destoop’s picture

nils.destoop CreditAttribution: nils.destoop commented
Status: Needs work » Closed (fixed)