aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* pg, `PostgreSQL::Name` to hold schema qualified names.Yves Senn2014-05-304-19/+96
|
* Merge pull request #15417 from JuanitoFatas/doc/semver-of-railsZachary Scott2014-05-291-4/+23
|\ | | | | [ci skip] Document Rails' shifted semver.
| * [ci skip] Document Rails' shifted semver.Juanito Fatas2014-05-301-4/+23
|/
* Mapping never actually uses @set, so rmAaron Patterson2014-05-292-4/+4
|
* no reason to make a Mapper object if the path is blankAaron Patterson2014-05-291-1/+2
|
* Merge branch 'mapper'Aaron Patterson2014-05-2912-210/+220
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * mapper: (34 commits) no more is_a checks on instantiation Path::Pattern is instantiated internally, so make the contructor require a strexp object Strexp#names is only used in a test, so rm pass the parsed path from mapper to the Strexp add an alternate constructor to Strexp that takes a string ask the strexp for the ast remove dead code disconnect path from the instance reuse the ast we already made use a parser to extract the group parts from the path pass the parsed parameters through the methods so we don't reparse or require caching code "controllers" should be a valid path name controllers with slash names are also not supported, so we can reuse the message only validate controllers golf down a bit only error handling between controller and action is the same add a test for controllers without colons move nil check to a method that yields to a block if the value is not nil translate action / controller to the desired object only one nil check on the action variable ...
| * no more is_a checks on instantiationAaron Patterson2014-05-292-13/+4
| |
| * Path::Pattern is instantiated internally, so make the contructor require a ↵Aaron Patterson2014-05-295-46/+49
| | | | | | | | strexp object
| * Strexp#names is only used in a test, so rmAaron Patterson2014-05-292-36/+0
| |
| * pass the parsed path from mapper to the StrexpAaron Patterson2014-05-292-5/+9
| |
| * add an alternate constructor to Strexp that takes a stringAaron Patterson2014-05-296-39/+41
| |
| * ask the strexp for the astAaron Patterson2014-05-292-3/+7
| |
| * remove dead codeAaron Patterson2014-05-291-1/+0
| |
| * disconnect path from the instanceAaron Patterson2014-05-291-6/+6
| |
| * reuse the ast we already madeAaron Patterson2014-05-291-9/+10
| |
| * use a parser to extract the group parts from the pathAaron Patterson2014-05-293-5/+9
| |
| * pass the parsed parameters through the methods so we don't reparse orAaron Patterson2014-05-291-38/+34
| | | | | | | | require caching code
| * Merge branch 'master' into mapperAaron Patterson2014-05-2922-73/+148
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: Update url to rake docs [ci skip] Name#model_name doesn't return a String object Result sets never override a model's column type [ci skip] Make last note show up in postgresql guide. Add missing `:param` option from the docs for Mapper#match [ci skip] Option discovered by @zackperdue in #14741, implemented in #5581. Add @senny's changed from #14741, including code font for `resources` options, and wrapped to 80 chars. [ci skip] Use github url for homepage of log4r [ci skip] Remove TODO. Ensure we always use instances of the adapter specific column class Fix indentation from 1b4b26f [ci skip] [ci skip] Improve form_helpers.md guide. Clear inflections after test. Remove unnecessary include for integration tests. Added documentation for the :param option for resourceful routing
| * | "controllers" should be a valid path nameAaron Patterson2014-05-292-1/+11
| | |
| * | controllers with slash names are also not supported, so we can reuse the messageAaron Patterson2014-05-281-6/+2
| | |
| * | only validate controllersAaron Patterson2014-05-281-15/+17
| | |
| * | golf down a bitAaron Patterson2014-05-281-8/+4
| | |
| * | only error handling between controller and action is the sameAaron Patterson2014-05-281-18/+16
| | |
| * | add a test for controllers without colonsAaron Patterson2014-05-281-0/+10
| | |
| * | move nil check to a method that yields to a block if the value is not nilAaron Patterson2014-05-281-17/+16
| | |
| * | translate action / controller to the desired objectAaron Patterson2014-05-281-12/+11
| | |
| * | only one nil check on the action variableAaron Patterson2014-05-281-10/+8
| | |
| * | change to case / when on typesAaron Patterson2014-05-281-8/+7
| | |
| * | only do one nil check against the controllerAaron Patterson2014-05-281-9/+9
| | |
| * | extract controller and action parsing to a methodAaron Patterson2014-05-281-19/+26
| | |
| * | we don't need the call to presence. that is my present, to you!Aaron Patterson2014-05-281-1/+1
| | |
| * | swtich to returning early if to responds to callAaron Patterson2014-05-281-35/+32
| | |
| * | return early if we have a valid controller nameAaron Patterson2014-05-281-5/+5
| | |
| * | trade 2 is_a? checks for a nil checkAaron Patterson2014-05-281-2/+4
| | |
| * | invert logic to remove nil? and exclude? checks (use ruby rather than AS ↵Aaron Patterson2014-05-281-6/+6
| | | | | | | | | | | | when possible
| * | fewer blank? callsAaron Patterson2014-05-281-4/+4
| | |
| * | reduce action.blank? callsAaron Patterson2014-05-281-4/+6
| | |
| * | reduce blank? checksAaron Patterson2014-05-281-4/+9
| | |
| * | extract controller checks to methodsAaron Patterson2014-05-281-19/+26
| | |
| * | set defaults at the top so we can avoid the ||= testAaron Patterson2014-05-281-6/+9
| | |
| * | add a test for existing mapper functionalityAaron Patterson2014-05-281-0/+12
| | | | | | | | | | | | | | | I'm not sure if this is actually used, but I'm adding a test to define the behavior
* | | Merge pull request #15414 from akshay-vishnoi/doc_changesRafael Mendonça França2014-05-291-2/+3
|\ \ \ | | | | | | | | [ci skip] Include ActiveModel::Model in a class instead of inheriting
| * | | [ci skip] Include ActiveModel::Model in a class instead of inheritingAkshay Vishnoi2014-05-301-2/+3
|/ / /
* | | Generated engines don't include bin/rake, fixes #15409 [ci skip]Zachary Scott2014-05-291-5/+5
| | |
* | | Merge pull request #15413 from sgrif/sg-i-wish-i-couldRafael Mendonça França2014-05-291-2/+0
|\ \ \ | | | | | | | | Don't reference comments that do not exist
| * | | Don't reference comments that do not existSean Griffin2014-05-291-2/+0
|/ / /
* | | Merge pull request #15412 from sgrif/sg-serialized-type-cast-for-writeRafael Mendonça França2014-05-296-86/+47
|\ \ \ | | | | | | | | Move `type_cast_for_write` behavior over to the serialized type object
| * | | Move `type_cast_for_write` behavior over to the serialized type objectSean Griffin2014-05-296-86/+47
|/ / /
* | | Add missing period from 4b802bc [ci skip]Zachary Scott2014-05-291-1/+1
| | |
* | | Merge pull request #15411 from nathany/patch-1Zachary Scott2014-05-291-1/+1
|\ \ \ | | | | | | | | Fix broken link in Rails 3.0 release_notes [ci skip]