Core doesn't use this docblock tag -- another stowaway crept in from PHPStorm probably.

CommentFileSizeAuthor
#3 flag-remove_package_from_docblock-2629162-3.patch2.16 KBzerolab
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joachim created an issue. See original summary.

socketwench’s picture

socketwench CreditAttribution: socketwench as a volunteer commented
Issue tags: +Novice
zerolab’s picture

zerolab CreditAttribution: zerolab at Torchbox commented
Status: Active » Needs review
FileSize
flag-remove_package_from_docblock-2629162-3.patch2.16 KB

Removed all @package references.

socketwench’s picture

socketwench CreditAttribution: socketwench as a volunteer commented
Status: Needs review » Reviewed & tested by the community

Looks good, applies clean, and I can find no further instances of @package in the module.

martin107’s picture

martin107 CreditAttribution: martin107 commented

TLDR

console generate:service

dumps the @package into the new service.

joachim’s picture

joachim CreditAttribution: joachim commented

Hmm that's a thought, I should add that to Module Builder :)

  • joachim committed 8e3a528 on 8.x-4.x authored by zerolab 
    Issue #2629162 by zerolab: Removed @package from docblocks.
joachim’s picture

joachim CreditAttribution: joachim commented
Status: Reviewed & tested by the community » Fixed

Committed.

Thanks for the patch!

Status: Fixed » Closed (fixed)

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