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

	Commit message (Collapse)	Author	Age	Files	Lines
...
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26144 from javierjulio/patch-1	Jon Moss	2016-08-12	1	-1/+1
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|_\|/ / / / / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Update example in Http::Headers object to use `.from_hash`
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Given a hash (Rails 5) .from_hash must be used	Javier Julio	2016-08-12	1	-1/+1
\|/ / / / / / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	When initializing an `ActionDispatch::Http::Headers` object it takes a request object (Rails 5) whereas before it took a hash (Rails 4.x) but the documented example still shows a hash given to the constructor (due to commit 34fa6658dd1b779b21e586f01ee64c6f59ca1537) so this is just a documentation change to use the new `from_hash` method introduced in that earlier commit.
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26133 from smellsblue/dont-fail-on-non-string	Rafael França	2016-08-12	2	-1/+9
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Ensure values are strings before calling gsub
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Ensure values are strings before calling gsub	Mike Virata-Stone	2016-08-12	2	-1/+9
\|/ / / / / / / / / / / / / / / /
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26125 from qinix/fix-doc	Jon Moss	2016-08-11	1	-3/+3
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	[ci skip] Fix the wrong ActionCable documentation in the guide.
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	[ci skip] Fix documentation wrong for ActionCable	Eric Zhang	2016-08-12	1	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	[ci skip] Fix documentation for ActionCable::Channel#broadcast_to	Eric Zhang	2016-08-12	1	-2/+2
\| \| \|_\|_\|_\|_\|_\|/ / / / / / / / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26118 from alexcameron89/param-encoding-documentation	Jon Moss	2016-08-11	1	-0/+1
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|_\|_\|/ / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	[ci skip] Add documentation to Parameter Encoding
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	[ci skip] Add documentation to Parameter Encoding	Alex Kitchens	2016-08-11	1	-0/+1
\| \|/ / / / / / / / / / / / / /
* \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Integration test to prevent regression for the 5th time	Aaron Patterson	2016-08-11	1	-0/+47
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix unsafe query generation risk. Redo of CVE-2012-2660, CVE-2012-2694 and CVE-2013-0155 CVE-2016-6317
* \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26124 from yahonda/diag26099	Sean Griffin	2016-08-11	1	-4/+2
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Creating a new Topic class instead of class_eval for the existing one
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Creating a new Topic class instead of class_eval for the existing one	Yasuo Honda	2016-08-11	1	-4/+2
\| \|/ / / / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	since it affects another test `ReflectionTest#test_read_attribute_names` Address #26099
* \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26090 from kamipo/fix_warning_ambiguous_first_argument	Sean Griffin	2016-08-11	1	-1/+1
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix warning: ambiguous first argument
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix warning: ambiguous first argument	Ryuta Kamizono	2016-08-12	1	-1/+1
\|/ / / / / / / / / / / / / / /
* / / / / / / / / / / / / / /	ensure tag/content_tag escapes " in attribute vals	Andrew Carpenter	2016-08-11	2	-1/+11
\|/ / / / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Many helpers mark content as HTML-safe without escaping double quotes -- including `sanitize`. Regardless of whether or not the attribute values are HTML-escaped, we want to be sure they don't include double quotes, as that can cause XSS issues. For example: `content_tag(:div, "foo", title: sanitize('" onmouseover="alert(1);//'))` CVE-2016-6316
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26116 from y-yagi/fix_name_error_in_engine_commands_tasks	Kasper Timm Hansen	2016-08-11	2	-0/+26
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|_\|/ / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \|	add missing require rake
\| * \| \| \| \| \| \| \| \| \| \| \| \|	add missing require rake	yuuji.yaginuma	2016-08-11	2	-0/+26
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	In ff8035dfeed8c86594c32ef8e9204806e190cb58, require rake is deferred. Therefore, it is necessary to require rake even `Engine::CommandsTasks.
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26114 from kamipo/fix_test_failure	Kasper Timm Hansen	2016-08-11	2	-3/+3
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \|	Fix actionview test failure
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Fix actionview test failure	Ryuta Kamizono	2016-08-11	2	-3/+3
\|/ / / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Caused by #26092.
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26110 from aditya-kapoor/minor-doc-fix	Guillermo Iguaran	2016-08-10	1	-2/+5
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Minor doc fix related to ActiveModel::SecurePassword [ci skip]
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Minor doc fix related to ActiveModel::SecurePassword [ci skip]	Aditya Kapoor	2016-08-11	1	-2/+5
\| \| \|_\|_\|_\|/ / / / / / / / \| \|/\| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \|	prefer __dir__ over __FILE__ in File.expand_path	Xavier Noria	2016-08-11	6	-6/+18
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Thinking .. relative to files is not natural, we are used to think "parent of a directory", and we have __dir__ nowadays.
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #25930 from mechanicles/doc-http-cache-forever	Vipul A M	2016-08-10	1	-0/+24
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add documentation for `http_cache_forever`. [ci skip]
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Add documentation for `http_cache_forever`. [ci skip]	Santosh Wadghule	2016-08-10	1	-0/+24
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26092 from kerrizor/kerrizor/force-param-encoding	Aaron Patterson	2016-08-10	10	-20/+139
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Allow specifying encoding of parameters by action
\| * \| \| \| \| \| \| \| \| \| \| \| \| \|	Allow specifying encoding of parameters by action	Kerri Miller	2016-08-09	10	-20/+139
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	At GitHub we need to handle parameter encodings that are not UTF-8. This patch allows us to specify encodings per parameter per action.
* \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26102 from gsamokovarov/schema-statements-typo	Yves Senn	2016-08-10	2	-1/+7
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix a NoMethodError schema_statements.rb
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix a NoMethodError schema_statements.rb	Genadi Samokovarov	2016-08-10	2	-1/+7
\| \| \|/ / / / / / / / / / / / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	If you call `remove_index` with wrong options, say a type, like I did, you get: ``` == 20160810072541 RemoveUniqueIndexOnGoals: migrating ========================= -- remove_index(:goal, {:coulmn=>:kid_id, :unique=>true}) rails aborted! StandardError: An error has occurred, this and all later migrations canceled: undefined method `ArgumentError' for #<ActiveRecord::ConnectionAdapters::PostgreSQLAdapter:0x007fb7dec91b28> ``` What happened is that I mistyped column (coulmn) and got a `NoMethodError`, because of a missing comma during the raise. This made Ruby think we're calling the method `ArgumentError`.
* / \| \| \| \| \| \| \| \| \| \| \| \| \|	Remove unused method	Andrew White	2016-08-10	1	-5/+0
\|/ / / / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	In c546a2b parameter handling in AC test cases was changed to round tripping through encoders/decoders so that they matched reality and in 0adb8f8 the old methods were removed but the `html_format?` method was overlooked.
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26095 from kamipo/fix_broken_alignment_by_auto_correct	Xavier Noria	2016-08-09	20	-39/+77
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|_\|/ / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \|	Fix broken alignments caused by auto-correct commit 411ccbd
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Fix broken alignments caused by auto-correct commit 411ccbd	Ryuta Kamizono	2016-08-10	20	-39/+77
\|/ / / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Hash syntax auto-correcting breaks alignments. 411ccbdab2608c62aabdb320d52cb02d446bb39c
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #24476 from vipulnsward/24314-eagerloading-doc-update	Vipul A M	2016-08-09	1	-2/+3
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Document know limitation about using `references` in conjunction with custom select clauses [ci skip]
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Document know limitation about using `references` in conjunction with eager ↵	Vipul A M	2016-08-09	1	-2/+3
\| \| \|/ / / / / / / / / / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	loading causing it to ignore custom select clauses. [ci skip] Fixes #24314
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26042 from bogdanvlviv/debugging_guide	Vipul A M	2016-08-09	1	-0/+1
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \|	Add link to 'Pry' in debugging guide [ci skip]
\| * \| \| \| \| \| \| \| \| \| \| \|	Add link to 'Pry' in debugging guide [ci skip]	bogdanvlviv	2016-08-07	1	-0/+1
\| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26086 from y-yagi/update_rescue_responses_default	Jon Moss	2016-08-08	1	-0/+2
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \|_\|/ / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \|	update list of rescue_responses default [ci skip]
\| * \| \| \| \| \| \| \| \| \| \| \|	update list of rescue_responses default [ci skip]	yuuji.yaginuma	2016-08-09	1	-0/+2
\|/ / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Follow up to fe859a54219740fa8b4e09a592820d2ee12ba222
* \| \| \| \| \| \| \| \| \| \| \|	revises more Lint/EndAlignment offenses	Xavier Noria	2016-08-08	8	-30/+30
\| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26067 from yahonda/oracle_can_use_fetch_first	Yves Senn	2016-08-08	2	-4/+4
\|\ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Use `FETCH FIRST` for Oracle12 and test `ROWNUM <=` for Oracle 11g or older version to test sql limit behavior
\| * \| \| \| \| \| \| \| \| \| \| \|	Use `FETCH FIRST` for Oracle12 database and Arel Oracle12 visitor	Yasuo Honda	2016-08-08	2	-4/+4
\|/ / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	also test `ROWNUM <=` for Oracle 11g or older version of Oracle and Oracle visitor Oracle 12c database and Arel Oracle12 visitor supports better top N query.
* \| \| \| \| \| \| \| \| \| \| \|	let instance thread_mattr_* methods delegate to the class-level ones	Xavier Noria	2016-08-08	1	-4/+10
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This code has too much duplication and the rationale for the concatenation may not be obvious to the reader. You define the ones at class-level, explain why does the code concatenates there, and then the convenience ones at instance-level just delegate.
* \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #25681 from willnet/fix-thread_mattr_accessor	Yves Senn	2016-08-08	3	-4/+48
\|\ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix `thread_mattr_accessor` share variable superclass with subclass
\| * \| \| \| \| \| \| \| \| \| \| \|	Fix `thread_mattr_accessor` share variable superclass with subclass	willnet	2016-08-04	3	-8/+35
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The current implementation of `thread_mattr_accessor` set variable sharing superclass with subclass. So the method doesn't work as documented. Precondition class Account thread_mattr_accessor :user end class Customer < Account end Account.user = "DHH" Account.user #=> "DHH" Customer.user = "Rafael" Customer.user # => "Rafael" Documented behavior Account.user # => "DHH" Actual behavior Account.user # => "Rafael" Current implementation set variable statically likes `Thread[:attr_Account_user]`, and customer also use it. Make variable name dynamic to use own thread-local variable.
* \| \| \| \| \| \| \| \| \| \| \| \|	damn typos [ci skip]	Xavier Noria	2016-08-08	1	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \|	explain why aliasing uses explicit selfs [ci skip]	Xavier Noria	2016-08-08	1	-0/+3
\| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \|	code gardening: removes redundant selfs	Xavier Noria	2016-08-08	74	-122/+121
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	A few have been left for aesthetic reasons, but have made a pass and removed most of them. Note that if the method `foo` returns an array, `foo << 1` is a regular push, nothing to do with assignments, so no self required.
* \| \| \| \| \| \| \| \| \| \| \| \|	revises most Lint/EndAlignment offenses	Xavier Noria	2016-08-07	13	-55/+58
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Some case expressions remain, need to think about those ones.
* \| \| \| \| \| \| \| \| \| \| \| \|	applies project convention for string literals	Xavier Noria	2016-08-07	1	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \|	adds missing comma in assert call	Xavier Noria	2016-08-07	1	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \|	copy-edits an exception message	Xavier Noria	2016-08-07	1	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Inserted spaces in the name of Rails components. Since I was on it, also used PostgreSQL instead of Postgres because albeit Postgres is an accepted alias, PostgreSQL is the official name and the actual name of the adapter. See https://wiki.postgresql.org/wiki/ProjectName with regard to PostgreSQL vs Postgres.