aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | | Revert "Merge pull request #12208 from mjhoy/patch-1"Rafael Mendonça França2013-09-121-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit ab5cd54b7e791f8419f689d1bef5394890268a6f, reversing changes made to cdc10c898d4865302740340eedec4f5f4ca76565. Reason: This way of defining root path is still supported. See https://github.com/rails/rails/blob/d262773ab7f0aae5de2d354ac2eca168e95b653d/actionpack/test/controller/routing_test.rb#L450-457
* | | | | | | | | | Merge branch 'master' of github.com:rails/docrailsVijay Dev2013-09-1311-129/+293
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Wrap the guides in 80 columns [ci skip]Rafael Mendonça França2013-09-121-101/+260
| | | | | | | | | | |
| * | | | | | | | | | Merge pull request #163 from mogetutu/masterRafael Mendonça França2013-09-121-1/+1
| |\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Added quotes when defining the precision for the `decimal` fields
| | * | | | | | | | | | Added quotes when defining the precision for the `decimal` fieldsmogetutu2013-09-121-1/+1
| |/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | before: ```bash $ rails generate migration AddDetailsToProducts price:decimal{5,2} supplier:references{polymorphic} ``` after: ```bash $ rails generate migration AddDetailsToProducts 'price:decimal{5,2}' supplier:references{polymorphic} ```
| * | | | | | | | | | Plural of Deer not deerSergio2013-09-101-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change the plural of Deer word in the naming conventions
| * | | | | | | | | | html generated for a search formSergio2013-09-091-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There's a div that is generated in rails for a default search form and is not includen in the example.
| * | | | | | | | | | Change documentation to consistently refer to the same objectEarl J St Sauver2013-09-081-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The documentation in this section is referring to a profile, so the resource that's created should probably also be a profile of some sort.
| * | | | | | | | | | Blacklist refferenced in docs is actually whitelistEarl J St Sauver2013-09-081-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The docs refference a blacklist, but really what's being described is a whitelist. Anything that matches the constraint gets through to the path.
| * | | | | | | | | | added # or commentSergio2013-09-051-1/+1
| | | | | | | | | | |
| * | | | | | | | | | Review the changes made on 90155b4e28a3887dce9428e9df150ede3d6c7465Rafael Mendonça França2013-09-041-5/+9
| | | | | | | | | | |
| * | | | | | | | | | Client.exists?(1,2,3) and Client.exists?([1,2,3]) does not workSergio2013-09-051-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The right command for doing that is Client.exists?id:[1,2,3] Exists does not work like find method, in find method you can do Person.find(1, 2, 6) or Person.find([7, 17]) but not Person.exists?(1,2,3) or Person.exists?([1,2,3])
| * | | | | | | | | | Example of Join modelsSergio2013-09-041-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the example of join models, there are five models but in the explanation only consider Category, Post, Comment and Guest.
| * | | | | | | | | | `implemention` => `implementation`Vipul A M2013-09-031-1/+1
| | | | | | | | | | |
| * | | | | | | | | | `attibutes` => `attributes`Vipul A M2013-09-031-1/+1
| | | | | | | | | | |
| * | | | | | | | | | Be assertive about adding `[ci skip]` messageVipul A M2013-09-031-1/+1
| | | | | | | | | | |
| * | | | | | | | | | 4.0 Release notes should point to 4.0 ChangelogVipul A M2013-09-031-6/+6
| | | | | | | | | | |
| * | | | | | | | | | Fix spacing issueVipul A M2013-09-031-0/+1
| | | | | | | | | | |
* | | | | | | | | | | 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.
generated by cgit v1.2.3 (git 2.25.1) at 2024-08-27 10:32:42 +0000