aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #12208 from mjhoy/patch-1Rafael Mendonça França2013-09-121-4/+0
|\ | | | | remove outdated docs
| * remove outdated docsMichael Hoy2013-09-121-4/+0
|/
* Merge pull request #12196 from h-lame/fix-activesupport-cache-filestore-cleanupRafael Mendonça França2013-09-123-1/+18
|\ | | | | Fix FileStore#cleanup to no longer rely on missing each_key method
| * Fix FileStore#cleanup to no longer rely on missing each_key methodMurray Steele2013-09-113-1/+18
| |
* | Merge pull request #12207 from chancancode/fix_json_loadRafael Mendonça França2013-09-122-2/+14
|\ \ | | | | | | Enabled quirks mode on JSON.parse, fixes broken test in af9caae
| * | Enabled quirks mode on JSON.parse, fixes broken test in af9caaeGodfrey Chan2013-09-122-2/+14
|/ / | | | | | | | | | | | | It turns out that ActionPack depends on the decoder to parse JSON "fragments" (e.g. '"a string"', '1', 'null', etc), so we need to enable quirks mode on JSON.parse. Also added coverage on the decoder side to prevent regression.
* | Write the right CHANGELOG entry for #11603Rafael Mendonça França2013-09-121-1/+1
| | | | | | | | [ci skip]
* | Merge pull request #11603 from ↵Rafael Mendonça França2013-09-123-2/+10
|\ \ | | | | | | | | | | | | | | | jetthoughts/join_strings_instead_of_contactination Cleanup: replace String concatenation by joining for excerpt helper
| * | Cleanup of excerpt helperPaul Nikitochkin2013-09-063-2/+10
| | | | | | | | | | | | | | | * replaced String concatenation by joining * separator has default value to '', even it is nil
* | | Merge pull request #12206 from harshadsabne/masterRafael Mendonça França2013-09-121-3/+3
|\ \ \ | | | | | | | | Update 4_0_release_notes.md
| * | | Update 4_0_release_notes.mdHarshad Sabne2013-09-121-3/+3
|/ / / | | | | | | Active support --> Deprecations Code highlighted.
* | | Disable prepared statements in the unprepared_statement blockRafael Mendonça França2013-09-121-2/+3
| | |
* | | Fix typo on instance variable get callCarlos Antonio da Silva2013-09-121-1/+1
| | |
* | | Merge pull request #12205 from claudiob/remove-unused-raw-email-fixturesCarlos Antonio da Silva2013-09-1215-736/+0
|\ \ \ | | | | | | | | Remove unused raw email fixtures
| * | | Remove unused raw email fixturesclaudiob2013-09-1215-736/+0
|/ / / | | | | | | | | | The tests that used the raw_email_* fixtures were removed in d500ad3
* | | Merge pull request #12201 from chancancode/json_loadJeremy Kemper2013-09-112-3/+16
|\ \ \ | | | | | | | | Replace JSON.load with JSON.parse
| * | | Replace JSON.load with JSON.parse, also removed the proc parameterGodfrey Chan2013-09-112-3/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since we are dealing with untrusted user input, we should not be using JSON.load. According to the docs[1]: BEWARE: This method is meant to serialise data from trusted user input, like from your own database server or clients under your control, it could be dangerous to allow untrusted users to pass JSON sources into it. The default options for the parser can be changed via the ::load_default_options method. [1] http://www.ruby-doc.org/stdlib-2.0/libdoc/json/rdoc/JSON.html#method-i-load
* | | | these are not real developer objects, so counting them doesn't makeAaron Patterson2013-09-111-1/+1
| | | | | | | | | | | | | | | | | | | | sense. Let's load the object to ensure it's an array and count the array.
* | | | Remove conditional adding a new methodRafael Mendonça França2013-09-112-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | At the point we want to add the reflection we already know the reflection is of the AggregateReflection type so we can call a specific method
* | | | Fix inverted conditionalRafael Mendonça França2013-09-111-1/+1
| | | | | | | | | | | | | | | | Oops, I should had run the tests
* | | | Remove invalid commentRafael Mendonça França2013-09-111-4/+0
| | | | | | | | | | | | | | | | This is not valid anymore after 08477a651648ba4417ded128aa37b9ae0dcbc9ce
* | | | Check if the SQL is not a prepared statementRafael Mendonça França2013-09-1110-7/+29
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | When the adapter is with prepared statement disabled and the binds array is not empty the connection adapter will try to set the binds values and will fail. Now we are checking if the adapter has the prepared statement disabled. Fixes #12023
* | | Merge pull request #12188 from SamSaffron/masterAaron Patterson2013-09-114-12/+16
|\ \ \ | | | | | | | | Perf: avoid dupes add fallback logic for coders
| * | | Perf: avoid dupes add fallback logic for codersSam2013-09-114-12/+16
| | | |
* | | | Merge pull request #12200 from dchelimsky/simplify-duration-inspect-even-moreRafael Mendonça França2013-09-112-6/+7
|\ \ \ \ | | | | | | | | | | Reduce Duration#inspect to a single series of transformations
| * | | | Reduce Duration#inspect to a single series of transformationsDavid Chelimsky2013-09-112-6/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * eliminates need for temp Hash Also added a couple of examples to DurationTest to specify: * duration can be defined with units out of order e.g. 1.month + 1.year + 1.second + 1.day * equality with a Fixnum works regardless of which operand is on which side of the operator
* | | | | Merge pull request #12135 from dylanahsmith/avoid_empty_transactionRafael Mendonça França2013-09-113-1/+10
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Avoid empty transaction from setting has_one association on new record. Conflicts: activerecord/CHANGELOG.md
| * | | | | Avoid empty transaction from setting has_one association on new record.Dylan Thacker-Smith2013-09-113-1/+9
| | | | | |
* | | | | | WhitespacesRafael Mendonça França2013-09-111-1/+2
| | | | | |
* | | | | | Merge pull request #12185 from SamSaffron/join_depRafael Mendonça França2013-09-111-1/+13
|\ \ \ \ \ \ | | | | | | | | | | | | | | Reduce allocations when extracting AR models
| * | | | | | Reduce allocations when extracting AR modelsSam2013-09-111-1/+13
| | | | | | |
* | | | | | | Merge pull request #12194 from thedarkone/readonly-merger-fixRafael Mendonça França2013-09-113-1/+17
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Relation#merge should not lose readonly(false) flag.
| * | | | | | | Relation#merge should not lose readonly(false) flag.thedarkone2013-09-113-1/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The original code ignores the `false` value because `false.blank? # => true`.
* | | | | | | | Revert "Add meta tag with charset information to application layout."Aaron Patterson2013-09-111-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit ba0407337e93c4ef55cef3472143f62e8a984a64.
* | | | | | | | fix deleting join models with no pkAaron Patterson2013-09-112-6/+62
| |_|/ / / / / |/| | | | | |
* | | | | | | Merge pull request #12199 from michaelminter/masterRafael Mendonça França2013-09-111-1/+1
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | | remove sentence err
| * | | | | | remove sentence errmichaelminter2013-09-111-1/+1
|/ / / / / /
* | | | | | Merge pull request #12184 from waynn/patch-4Steve Klabnik2013-09-101-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | "previous version of Rails" is gramatically incorrect
| * | | | | | [ci skip] 'previous version of Rails' is gramatically incorrectWaynn Lue2013-09-101-1/+1
| | |/ / / / | |/| | | |
* | | | | | Add meta tag with charset information to application layout.Steve Klabnik2013-09-101-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, our default HTML would validate properly, but would generate a warning: it doesn't declare a character encoding. According to [the spec][encoding-spec], if you don't specify an encoding, a 7 step algorithm happens, with a toooon of sub-steps. Or, we could just actually specify it. Since everything else in Rails assumes UTF-8, we should make sure pages are served with that encoding too. This meta tag is the simplest way to accomplish this. More resources: * http://blog.whatwg.org/the-road-to-html-5-character-encoding * http://www.w3.org/International/tutorials/tutorial-char-enc/ * http://validator.w3.org/ [encoding-spec]: http://www.whatwg.org/specs/web-apps/current-work/multipage/parsing.html#determining-the-character-encoding
* | | | | | add a comment for sanity of other people to comeAaron Patterson2013-09-101-0/+8
| | | | | |
* | | | | | ask the association for records rather than calling `send`Aaron Patterson2013-09-101-1/+2
| | | | | |
* | | | | | Merge pull request #12193 from arunagw/revert-revertRafael Mendonça França2013-09-1019-41/+33
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Revert fixes
| * | | | | Use Ruby 2.0 caller_locations instead of caller if availableAkira Matsuda2013-09-101-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * we no more have to manipulate the each caller strings by ourselves using caller_locations * caller_locations runs slightly faster, and creates less objects than good old caller Benchmark (loading an Engine 1000 times): caller: 262.89 ms caller_locations: 186.068 ms
| * | | | | Update Rails 3.2.x guide link [ci skip]Tomas Varaneckas2013-09-101-1/+1
| | | | | |
| * | | | | Be sure to restore the default I18n.locale after changed its value in a testAkira Matsuda2013-09-101-3/+3
| | | | | |
| * | | | | Fixes typo in Object#try!Jay Hayes2013-09-101-1/+1
| | | | | |
| * | | | | More unused associations in AR test modelsAkira Matsuda2013-09-108-13/+0
| | | | | |
| * | | | | :scissors: [ci skip]Carlos Antonio da Silva2013-09-101-1/+1
| | | | | |
| * | | | | change function def self.table_name to self.table_nameRajarshi Das2013-09-102-5/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | change def self.primary_key to self.primary_key change def self.primary_key to self.primary_key