New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release v2.7.26 #22263

Merged
merged 3 commits into from Apr 4, 2017

Conversation

Projects
None yet
5 participants
@fabpot
Member

fabpot commented Apr 4, 2017

Changelog (since v2.7.25...v2.7.26)

  • bug #22229 [ExpressionLanguage] Provide the expression in syntax errors (@k0pernikus, @stof)
  • bug #22240 [DI] Fix fatal error at ContainerBuilder::compile() if config is not installed (@chalasr)
  • bug #22140 [Form] Improve the exceptions when trying to get the data in a PRE_SET_DATA listener and the data has not already been set (@fancyweb)
  • bug #22217 [Console] Fix table cell styling (@ro0NL)
  • bug #22194 [Console] CommandTester: disable color support detection (@julienfalque)
  • bug #22188 [Console] Revised exception rendering (@ro0NL)
  • bug #22154 [WebProfilerBundle] Normalize whitespace in exceptions passed in headers (@curry684)
  • bug #22142 [Console] Escape exception messages in renderException (@chalasr)
  • bug #22172 Fix port usage in server:status command (@alcaeus)
  • bug #22164 [Bridge\Doctrine] Fix change breaking doctrine-bundle test suite (@nicolas-grekas)
  • bug #22133 [Filesystem] normalize paths before making them relative (@xabbuh)
  • bug #22138 [HttpFoundation][bugfix] $bags should always be initialized (@MacDada)
  • bug #21810 #21809 [SecurityBundle] bugfix: if security provider's name contains upper cases then container didn't compile (@Antanas Arvasevicius)
  • bug #19778 [Security] Fixed roles serialization on token from user object (@eko)
  • bug #22022 [Validator] fix URL validator to detect non supported chars according to RFC 3986 (@e-moe)
  • bug #21968 Fixed pathinfo calculation for requests starting with a question mark. (@syzygymsu)
  • bug #21846 [HttpFoundation] Fix Request::getHost() when having several hosts in X_FORWARDED_HOST (@nicolas-grekas)
  • bug #21208 [Validator] Add object handling of invalid constraints in Composite (@SenseException)
  • bug #22044 [Serializer] [XML] Ignore Process Instruction (@jordscream)
  • bug #22079 [HttpKernel] Fixed bug with purging of HTTPS URLs (@ausi)
  • bug #21523 #20411 fix Yaml parsing for very long quoted strings (@RichardBradley)
  • bug #22001 [Doctrine Bridge] fix priority for doctrine event listeners (@dmaicher)
  • bug #21981 [Console] Use proper line endings in BufferedOutput (@julienfalque)
  • bug #21957 [Form] Choice type int values (BC Fix) (@mcfedr)
  • bug #21923 [travis] Test with hhvm 3.18 (@nicolas-grekas)
  • bug #21823 dumpFile(), preserve existing file permissions (@chs2)
  • bug #21865 [Security] context listener: hardening user provider handling (@xabbuh)
  • bug #21883 [HttpKernel] fix Kernel name when stored in a directory starting with a number (@fabpot)

@fabpot fabpot merged commit 1034146 into symfony:2.7 Apr 4, 2017

1 check was pending

fabbot.io Doing some proofreading and checking your coding style.
Details

@fabpot fabpot deleted the fabpot:release-2.7.26 branch Apr 4, 2017

@discordier

This comment has been minimized.

Show comment
Hide comment
@discordier

discordier Apr 4, 2017

Broken encoding? should be Benoît Burnichon

Broken encoding? should be Benoît Burnichon

This comment has been minimized.

Show comment
Hide comment
@stof

stof Apr 4, 2017

Member

this is Github doing weird things when displaying the diff. The encoding is right when looking at the full file.

Member

stof replied Apr 4, 2017

this is Github doing weird things when displaying the diff. The encoding is right when looking at the full file.

This comment has been minimized.

Show comment
Hide comment
@discordier

discordier Apr 4, 2017

It is broken in this diff view, blob view and raw.
So no, this name appears to be broken, all other UTF8 chars are correct, simply the î here is not.

It is broken in this diff view, blob view and raw.
So no, this name appears to be broken, all other UTF8 chars are correct, simply the î here is not.

This comment has been minimized.

Show comment
Hide comment
@stof

stof Apr 4, 2017

Member

this is fine in the raw view you linked for me:
raw_unicode

And also in the blob view:
blob_unicode

Member

stof replied Apr 4, 2017

this is fine in the raw view you linked for me:
raw_unicode

And also in the blob view:
blob_unicode

This comment has been minimized.

Show comment
Hide comment
@discordier

discordier Apr 4, 2017

I just realized, your screenshot can not be from the section I gave you because the surrounding lines do not match.
This brought to me that we have Benoît Burnichon twice in the file, one time as: Benoît Burnichon (line 982) and another one as Benoît Burnichon (bburnichon) (line 434)

I just realized, your screenshot can not be from the section I gave you because the surrounding lines do not match.
This brought to me that we have Benoît Burnichon twice in the file, one time as: Benoît Burnichon (line 982) and another one as Benoît Burnichon (bburnichon) (line 434)

This comment has been minimized.

Show comment
Hide comment
@theofidry

theofidry Apr 4, 2017

Contributor

@stof indeed I can see your input, but I also see another entry with Benoît Burnichon in both links

Contributor

theofidry replied Apr 4, 2017

@stof indeed I can see your input, but I also see another entry with Benoît Burnichon in both links

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment