Skip to content
This repository has been archived by the owner on Jan 30, 2020. It is now read-only.

fix regression: MessageId having double brackets #156

Closed
wants to merge 1 commit into from

Conversation

glensc
Copy link
Contributor

@glensc glensc commented Jun 12, 2017

this got broken from #86 when header lazyloading was omitted

Expected :'<CALTvGe4_oYgf9WsYgauv7qXh2-6=KbPLExmJNG7fCs9B=1nOYg@mail.example.com>'
Actual   :'<<CALTvGe4_oYgf9WsYgauv7qXh2-6=KbPLExmJNG7fCs9B=1nOYg@mail.example.com>>'

glensc added a commit to eventum/zend-mail that referenced this pull request Jun 12, 2017
glensc added a commit to eventum/composer that referenced this pull request Jun 12, 2017
@@ -14,7 +14,7 @@

/**
* @group Zend_Mail
* @covers Zend\Mail\Header\MessageId<extended>
* @covers \Zend\Mail\Header\MessageId<extended>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for @Covers, already correct without \ prefix, revert it imo

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe @covers does need \ prefix

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added it because phpstorm did not complete the class otherwise

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -20,7 +20,7 @@

/**
* @group Zend_Mail
* @covers Zend\Mail\Storage\Message<extended>
* @covers \Zend\Mail\Storage\Message<extended>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for @Covers, already correct without \ prefix, revert it imo

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added it for ide (phpstorm) to be able to resolve class

@glensc glensc changed the title fix MessageId having double brackets fix regression: MessageId having double brackets Jun 13, 2017
@glensc glensc force-pushed the message-id-brackets branch from 529dd85 to 6e17e96 Compare July 21, 2017 16:22
this got broken from pull/86 when header lazyloading was omitted
@glensc glensc force-pushed the message-id-brackets branch from 6e17e96 to f1cb350 Compare July 21, 2017 16:23
Copy link
Member

@Ocramius Ocramius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@Ocramius Ocramius self-assigned this Mar 1, 2018
@Ocramius Ocramius added the bug label Mar 1, 2018
@Ocramius Ocramius added this to the 2.9.0 milestone Mar 1, 2018
Ocramius added a commit that referenced this pull request Mar 1, 2018
@Ocramius
Copy link
Member

Ocramius commented Mar 1, 2018

@glensc manually merged into d3010b2, thanks!

@Ocramius Ocramius closed this Mar 1, 2018
@glensc glensc deleted the message-id-brackets branch March 4, 2019 21:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants