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

Multiple headers of the same type #189

Merged
merged 1 commit into from
Nov 29, 2019
Merged

Multiple headers of the same type #189

merged 1 commit into from
Nov 29, 2019

Conversation

michalbundyra
Copy link
Member

Fixes behaviour of toString when added multiple headers of the same type
using addHeaderLine method. The behavious was correct when we adding
headers using header instance (via method addHeader).
In all cases behaviour of toString should be the same.

  • Are you fixing a bug?
    • Detail how the bug is invoked currently.
    • Detail the original, incorrect behavior.
    • Detail the new, expected behavior.
    • Base your feature on the master branch, and submit against that branch.
    • Add a regression test that demonstrates the bug, and proves the fix.
    • Add a CHANGELOG.md entry for the fix.

@michalbundyra michalbundyra added this to the 2.10.1 milestone Sep 28, 2019
Fixes behaviour of toString when added multiple headers of the same type
using addHeaderLine method. The behavious was correct when we adding
headers using header instance (via method addHeader).
In all cases behaviour of toString should be the same.
michalbundyra added a commit that referenced this pull request Nov 29, 2019
michalbundyra added a commit that referenced this pull request Nov 29, 2019
@michalbundyra michalbundyra merged commit 4accbc2 into zendframework:master Nov 29, 2019
michalbundyra added a commit that referenced this pull request Nov 29, 2019
@michalbundyra michalbundyra deleted the hotfix/multiple-headers branch November 29, 2019 19:51
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.

2 participants