aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* drop one more string allocationAaron Patterson2013-11-061-1/+1
|
* drop string allocations in the log subscriberAaron Patterson2013-11-061-2/+12
|
* calculate the ivars to remove in advance as a set and cache them in aAaron Patterson2013-11-063-12/+17
| | | | | | | constant. `view_assigns` can use the precalculated sets and remove instance variables without allocating any extra arrays
* use a set and reject to avoid array allocationsAaron Patterson2013-11-061-4/+11
|
* each_with_object on the view_assigns hashAaron Patterson2013-11-061-3/+1
|
* use slice to avoid range allocationAaron Patterson2013-11-061-1/+3
|
* these variables are also privateAaron Patterson2013-11-061-0/+1
|
* instance_variables returns symbols, so we should use symbols in our listAaron Patterson2013-11-061-1/+1
|
* Merge pull request #12789 from claudiob/humanize-without-capitalizingXavier Noria2013-11-067-15/+58
|\ | | | | Add +capitalize+ option to Inflector.humanize
| * Add +capitalize+ option to Inflector.humanizeclaudiob2013-11-067-15/+58
|/ | | | | | | So strings can be humanized without being capitalized: 'employee_salary'.humanize # => "Employee salary" 'employee_salary'.humanize(capitalize: false) # => "employee salary"
* Merge pull request #12788 from kylefritz/patch-1Rafael Mendonça França2013-11-061-2/+2
| | | | | | rdoc: favicon source shouldn't begin with a slash to reference asset pipeline resource [ci skip] Conflicts: actionview/lib/action_view/helpers/asset_tag_helper.rb
* Merge pull request #12783 from chancancode/eliminate_direct_json_gem_useJeremy Kemper2013-11-068-31/+32
|\ | | | | Eliminate `JSON.{parse,load,generate,dump}` and `def to_json`
| * Eliminate `JSON.{parse,load,generate,dump}` and `def to_json`Godfrey Chan2013-11-058-31/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | JSON.{dump,generate} offered by the JSON gem is not compatiable with Rails at the moment and can cause a lot of subtle bugs when passed certain data structures. This changed all direct usage of the JSON gem in internal Rails code to always go through AS::JSON.{decode,encode}. We also shouldn't be implementing `to_json` most of the time, and these occurances are replaced with an equivilent `as_json` implementation to avoid problems down the road. See [1] for all the juicy details. [1]: intridea/multi_json#138 (comment)
* | bring back constant to expose the enum mapping as HWIA.Yves Senn2013-11-062-4/+19
| |
* | update the AR enum docs to match the actual behavior. [ci skip]Yves Senn2013-11-061-5/+3
|/
* Fix AS changelog indent, remove tabs and extra spaces [ci skip]Carlos Antonio da Silva2013-11-061-2/+2
|
* Merge pull request #12782 from chancancode/fix_object_and_struct_as_jsonJeremy Kemper2013-11-053-5/+40
|\ | | | | Fixed Object#as_json and Struct#as_json with options
| * Fixed Object#as_json and Struct#as_json with optionsGodfrey Chan2013-11-053-5/+40
|/ | | | | | | | | | | | These methods now takes the same options as Hash#as_json, for example: struct = Struct.new(:foo, :bar).new struct.foo = "hello" struct.bar = "world" json = struct.as_json(only: [:foo]) # => {foo: "hello"} This is extracted from PR #11728 from @sergiocampama, see also the discussion in #11460.
* store enum mapping using `Strings` instead of `Symbols`.Yves Senn2013-11-052-3/+9
| | | | | This allows to assign both `String` and `Symbol` values to the enum without having to call `to_sym`, which is a security problem.
* direct enum assignment rasies ArgumentError for unknown values.Yves Senn2013-11-052-1/+18
|
* Merge pull request #12776 from derekprior/dp-i18n-doc-fixRafael Mendonça França2013-11-051-1/+1
|\ | | | | Clarify HTML Safe Translations
| * Clarify HTML Safe Translations [ci-skip]Derek Prior2013-11-051-1/+1
| | | | | | | | | | | | I think it's confusing to say "Use them in views without escaping." We use all keys in views without escaping - the escaping is done for us automatically _unless_ we call html_safe or the key ends in _html.
* | define enum methods inside a `Module` to make them overwritable.Yves Senn2013-11-053-13/+34
| |
* | Merge pull request #12390 from kennyj/rename_to_association_methodsRafael Mendonça França2013-11-054-7/+7
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | Renamed generated_feature_methods to generated_association_methods. Conflicts: activerecord/lib/active_record/associations/builder/association.rb activerecord/lib/active_record/associations/builder/singular_association.rb activerecord/test/cases/base_test.rb
| * | Renamed generated_feature_methods to generated_association_methods.kennyj2013-09-285-12/+12
| | |
* | | Remove private API testRafael Mendonça França2013-11-051-10/+0
| |/ |/|
* | Merge pull request #12773 from ptn/use_19_syntax_in_schema_testGuillermo Iguaran2013-11-051-1/+1
|\ \ | | | | | | Use 1.9 Hash syntax consistently
| * | Use 1.9 Hash syntax consistentlyPablo Torres2013-11-051-1/+1
|/ /
* | Merge pull request #12752 from notalex/expand_path_refactoringYves Senn2013-11-053-3/+3
|\ \ | | | | | | Refactor File.expand_path usage to remove additional File.join
| * | Refactor File.expand_path usage to remove additional File.joinAlex Johnson2013-11-053-3/+3
| | |
* | | ask the fixture set for the sql statementsAaron Patterson2013-11-042-11/+17
| | |
* | | The enum value constant isn't used, so rm it for now.Aaron Patterson2013-11-042-27/+7
| | |
* | | stop doing the same calculation over and overAaron Patterson2013-11-041-4/+5
| | |
* | | oops, template replay needs to happen after bundle. :orz:Aaron Patterson2013-11-041-1/+1
| | |
* | | application template respects Gemfile entriesAaron Patterson2013-11-042-0/+25
| | |
* | | gems can be added or skipped from the templateAaron Patterson2013-11-044-2/+92
| | |
* | | fix copy & paste test-case naming. [ci skip]Yves Senn2013-11-041-1/+1
| | |
* | | document `BACKTRACE` env var usage in testing guide. [ci skip]Yves Senn2013-11-041-0/+11
| | |
* | | Fix failures related to enum changesCarlos Antonio da Silva2013-11-041-6/+4
| | | | | | | | | | | | | | | | | | Commit 6e4a810f51b9ed2468b92247ef5568cc9d6dbcc0 changed the implementation from class_eval to define_method, but missed the access to the constant versus the access to the constant name.
* | | Convert schema file to ruby 1.9 style hashCarlos Antonio da Silva2013-11-041-214/+214
| | |
* | | Style fixes on test schema fileCarlos Antonio da Silva2013-11-041-6/+6
| | |
* | | Merge pull request #12755 from cbartlett/masterCarlos Antonio da Silva2013-11-041-1/+1
|\ \ \ | | | | | | | | Remove extra whitespace
| * | | Remove extra whitespaceColin Bartlett2013-11-041-1/+1
| | | | | | | | | | | | | | | | | | | | When generating an app with --skip_active_record, an extra line of whitespace was included unnecessarily.
* | | | Refactor logic to grab the max time of the list of timestamp names in #cache_keyCarlos Antonio da Silva2013-11-042-4/+4
| | | | | | | | | | | | | | | | | | | | Reuse the already existing logic used for grabbing this information from the updated columns.
* | | | Respect cache timestamp format when giving timestamps to #cache_keyCarlos Antonio da Silva2013-11-042-3/+4
|/ / /
* | | :scissors: [ci skip]Carlos Antonio da Silva2013-11-042-5/+2
| | |
* | | Merge pull request #12754 from ↵David Heinemeier Hansson2013-11-041-4/+12
|\ \ \ | | | | | | | | | | | | | | | | ck3g/replace-class-evals-with-define-method-for-enum Replace `class_eval` with `define_method` for enum
| * | | Replace `class_eval` with `define_method` for enumVitaly Tatarintsev2013-11-041-4/+12
| |/ / | | | | | | | | | https://github.com/rails/rails/commit/db41eb8a6ea88b854bf5cd11070ea4245e1639c5#commitcomment-4502683
* | | Fix typo in railties changelogCarlos Antonio da Silva2013-11-041-2/+2
| | | | | | | | | | | | Also improve the example to show as shell command [ci skip]
* | | Fix changelog indent [ci skip]Carlos Antonio da Silva2013-11-041-6/+6
|/ /
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-08 04:25:14 +0000