rails.git - Mirror of official rails repo with custom fixes.

	Commit message (Collapse)	Author	Age	Files	Lines
*	Use frozen string literal in actionpack/	Kir Shatrov	2017-07-29	1	-0/+2
\|
*	Revert "Merge pull request #29540 from kirs/rubocop-frozen-string"	Matthew Draper	2017-07-02	1	-1/+0
\| \| \| \| \|	This reverts commit 3420a14590c0e6915d8b6c242887f74adb4120f9, reversing changes made to afb66a5a598ce4ac74ad84b125a5abf046dcf5aa.
*	Enforce frozen string in Rubocop	Kir Shatrov	2017-07-01	1	-0/+1
\|
*	Add more rubocop rules about whitespaces	Rafael Mendonça França	2016-10-29	1	-2/+2
\|
*	improve error message when include assertions fail	Michael Grosser	2016-09-16	1	-2/+2
\| \| \| \| \| \|	assert [1, 3].includes?(2) fails with unhelpful "Asserting failed" message assert_includes [1, 3], 2 fails with "Expected [1, 3] to include 2" which makes it easier to debug and more obvious what went wrong
*	Missing key should throw KeyError	eileencodes	2016-08-26	1	-0/+6
\| \| \| \| \| \|	It should not throw a NameError, but should throw a KeyError. Fixes #26278
*	Add three new rubocop rules	Rafael Mendonça França	2016-08-16	1	-15/+15
\| \| \| \| \| \| \| \|	Style/SpaceBeforeBlockBraces Style/SpaceInsideBlockBraces Style/SpaceInsideHashLiteralBraces Fix all violations in the repository.
*	applies new string literal convention in actionpack/test	Xavier Noria	2016-08-06	1	-9/+9
\| \| \| \| \|	The current code base is not uniform. After some discussion, we have chosen to go with double quotes by default.
*	Introduce `Headers#add`. Move `Response#add_header` upstream.	Jeremy Daer	2015-10-03	1	-0/+18
\| \| \| \| \| \| \|	* Introduce `ActionDispatch::Http::Headers#add` to add a value to a multivalued header. * Move `Response#add_header` upstream: https://github.com/rack/rack/pull/957 * Match upstream `Response#have_header?` -> `#has_header?` name change.
*	pass a request object to the headers object	Aaron Patterson	2015-08-21	1	-1/+1
\|
*	move header allocation to a helper method	Aaron Patterson	2015-08-21	1	-5/+9
\| \| \| \| \|	I'm going to change the constructor, so it's easier to do it in one place.
*	HTTP::Headers#key? correctly converts	schneems	2014-05-07	1	-0/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Previously if you were looking for a given key, the header may incorrectly tell you that it did not exist even though it would return a valid value: ```ruby env = { "CONTENT_TYPE" => "text/plain" } headers = ActionDispatch::Http::Headers.new(env) headers["Content-Type"] # => "text/plain" headers.key?("Content-Type") # => false ``` This PR fixes that behavior by converting the key before checking for presence
*	`Http::Headers` directly modifies the passed environment.	Yves Senn	2013-03-15	1	-6/+16
\| \| \| \| \| \| \| \| \|	The env hash passed to `Http::Headers#new` must be in env format. Also be aware that the passed hash is modified directly. docs and test-cases for setting headers/env in functional tests. Follow up to #9700.
*	`Http::Headers` respects dotted env vars, symbols, headers with numbers.	Yves Senn	2013-03-13	1	-0/+27
\|
*	allow headers and env to be passed in `IntegrationTest`.	Yves Senn	2013-03-13	1	-1/+44
\| \| \| \|	Closes #6513.
*	refactor, `Http::Headers` stores headers in env notation	Yves Senn	2013-03-13	1	-12/+21
\| \| \| \| \|	Also: cleanup, use consistent syntax for `Http::Header` and test.
*	Http::Headers respects headers that are not prefixed with HTTP_	Yves Senn	2013-03-13	1	-6/+13
\|
*	add test for fetch with a block	Aaron Patterson	2012-10-18	1	-0/+4
\|
*	prefer composition over inheritence	Aaron Patterson	2012-10-18	1	-0/+16
\|
*	Enable ActionDispatch::Http::Headers to support fetch	Mark Turner	2012-05-02	1	-0/+5
\|
*	Move dispatch related tests into test/dispatch	Joshua Peek	2009-01-28	1	-0/+16