aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* documentation fixGaurav Sharma2014-05-241-2/+2
|
* Merge pull request #15298 from ShunsukeAida/fix_capitalizationRafael Mendonça França2014-05-241-1/+1
|\ | | | | fixed capitalization [ci skip]
| * fixed capitalizationShunsukeAida2014-05-251-1/+1
| |
* | Add FIXME note about the thor bugRafael Mendonça França2014-05-241-0/+2
| |
* | Merge pull request #9083 from bronson/patch-1Rafael Mendonça França2014-05-241-1/+4
|\ \ | |/ |/| | | avoid using alias in generators
| * avoid using alias in generatorsScott Bronson2013-01-251-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Ruby's alias produces public methods, causing a spurious Thor task to be created.  For example, this is the reason MigrationGenerator currently has two tasks:     > ActiveRecord::Generators::MigrationGenerator.all_tasks.keys     => ["singular_name", "create_migration_file"] singular_name was meant to be an attribute, not a task.  Because it's public, it gets called as a task every time the generator is invoked. The fix is to ensure all generator methods have the correct visibility.
* | Merge pull request #13988 from zzak/guides_configuring_reverse_proxyZachary Scott2014-05-241-4/+38
|\ \ | | | | | | [Guides] Add documentation for using Rails with a reverse proxy [ci skip]
| * | Add documentation for using Rails with a reverse proxyZachary Scott2014-05-241-4/+38
| | |
* | | Merge pull request #14986 from dlangevin/trailing-slash-url-generationRafael Mendonça França2014-05-243-7/+37
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Fixes URL generation with trailing_slash: true Conflicts: actionpack/lib/action_dispatch/http/url.rb
| * | | Fixes URL generation with trailing_slash: trueDan Langevin2014-05-223-9/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | URL generation with trailing_slash: true was adding a trailing slash after .:format Routes.draw do resources :bars end bars_url(trailing_slash: true, format: 'json') # => /bars.json/ This commit removes that extra trailing slash
* | | | pg, remove duplicated code.Yves Senn2014-05-241-4/+0
| | | |
* | | | pg, extract schema definitions into separate file.Yves Senn2014-05-242-135/+141
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This mirrors the layout of abstract adapter and puts the definitions inside the `PostgreSQL` namespace (no longer under the adapter namespace). /cc @kares
* | | | Revert "Merge pull request #15293 from ↵Yves Senn2014-05-241-16/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | schneems/schneems/generator-output-in-error" This reverts commit 7466d6a9f989a1a8f90899b79b34f1152e440b7c, reversing changes made to b850c4c540e8f0c15742f67c532f521a4a123655. This was an accidental merge. A new PR will follow. /cc @schneems
* | | | Merge pull request #15293 from schneems/schneems/generator-output-in-errorRichard Schneeman2014-05-241-12/+16
|\ \ \ \ | | | | | | | | | | print generators on failed generate
| * | | | print generators on failed generateschneems2014-05-241-12/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Let's say we just ran: ``` $ rails g migrate add_click_to_issue_assignment ``` We will get an error that looks like: ``` Could not find generator migrate. ``` This patch adds all existing migrations to the output to make it easier for a developer to find a valid migration. ``` Could not find generator "migrate". Please select a valid generator: Rails: assets controller generator helper integration_test mailer migration model resource scaffold scaffold_controller task ``` It would be nice to do some spelling detection and suggest alternatives, but for now this should help.
* | | | | Merge pull request #15278 from akshay-vishnoi/doc_changesAndrew White2014-05-241-4/+5
|\ \ \ \ \ | | | | | | | | | | | | [ci skip] Improve doc for ModuleConstMissing.guess_for_anonymous
| * | | | | [ci skip] Improve doc for ModuleConstMissing.guess_for_anonymousAkshay Vishnoi2014-05-241-4/+5
| | | | | |
* | | | | | Merge pull request #15290 from sgrif/sg-consolidate-zero-scale-decimalRafael Mendonça França2014-05-236-41/+42
|\ \ \ \ \ \ | | | | | | | | | | | | | | Remove special case in schema dumper for decimal without scale
| * | | | | | Remove special case in schema dumper for decimal without scaleSean Griffin2014-05-236-41/+42
| | | | | | |
* | | | | | | Merge pull request #15291 from francocatena/router-visualizer-fixRafael Mendonça França2014-05-231-2/+2
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fix router visualizer CSS and JS resources
| * | | | | | | Fix router visualizer CSS and JS resourcesFranco Catena2014-05-231-2/+2
| | | | | | | |
* | | | | | | | Constraints contructor should always return a Constraints objectAaron Patterson2014-05-231-9/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I know, it's crazy.
* | | | | | | | default value is never used, so make it requiredAaron Patterson2014-05-231-1/+1
| | | | | | | |
* | | | | | | | push options decomposition up so we can extractAaron Patterson2014-05-231-20/+13
|/ / / / / / /
* | | | | | | Merge pull request #15289 from zzak/future_port_c8ddb61Godfrey Chan2014-05-233-14/+14
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Future port c8ddb61 [ci skip]
| * | | | | | | Future port c8ddb61Zachary Scott2014-05-233-14/+14
| | | | | | | |
* | | | | | | | glob_param is never used, so rmAaron Patterson2014-05-233-10/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | this also changes the constructor. We don't need to pass more options than "defaults" (whatever defaults are, ugh. probably another hash of stupid stuff).
* | | | | | | | Merge pull request #15285 from ↵Rafael Mendonça França2014-05-232-9/+5
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | sergey-alekseev/remove-active-record-where-duplicated-condition Remove duplicated parameter check on #where!
| * | | | | | | Remove duplicated parameter check on #where!Sergey Alekseev2014-05-232-9/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It seems that #where! is not designed to be used as a chained where. See initial implementation at 8c2c60511beaad05a218e73c4918ab89fb1804f0. So, no need to check twice. We should not test #where! https://github.com/rails/rails/pull/15285#discussion_r13018316
* | | | | | | | Merge pull request #15283 from oismail91/improve_doc_for_select_tagRafael Mendonça França2014-05-231-2/+2
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | | update select_tag docs for customizing id
| * | | | | | | update docs to include html id for select_tagOmar Ismail2014-05-231-2/+2
| |/ / / / / /
* | | | | | | Merge pull request #15277 from sgrif/sg-schema-cache-refactorRafael Mendonça França2014-05-231-31/+11
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | Simplify the code in schema cache
| * | | | | | Simplify the code in schema cacheSean Griffin2014-05-231-31/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The use of default procs was unnessecary, made the code confusing to follow, and made marshalling needlessly complex.
* | | | | | | Merge pull request #15261 from hbin/missing_parametersRafael Mendonça França2014-05-231-1/+5
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Bring the missing parameters back.
| * | | | | | | Bring the missing parameters back.Bin Huang2014-05-231-1/+5
| | | | | | | |
* | | | | | | | use Proc.new to automatically do parameter checking for usAaron Patterson2014-05-231-3/+1
| | | | | | | |
* | | | | | | | Merge pull request #15282 from sgrif/sg-remove-column-primaryRafael Mendonça França2014-05-237-52/+10
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Remove `Column#primary`
| * | | | | | | | Remove `Column#primary`Sean Griffin2014-05-237-52/+10
| | |/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It appears to have been used at some point in the past, but is no longer used in any meaningful way. Whether a column is considered primary is a property of the model, not the schema/column. This also removes the need for yet another layer of caching of the model's schema, and we can leave that to the schema cache.
* | | | | | | | Merge pull request #15276 from kuldeepaggarwal/fix-array-toRafael Mendonça França2014-05-232-1/+5
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Array#to now accept negative position also.
| * | | | | | | | revert 96525d63Kuldeep Aggarwal2014-05-232-1/+5
| | |_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | `Array#to` is working for negative position
* | | | | | | | Merge pull request #15265 from danielmbarlow/masterRafael Mendonça França2014-05-231-2/+3
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Added force_reload to Auto-generated methods doc
| * | | | | | | | Added force_reload to Auto-generated methods docdanielmbarlow2014-05-231-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | Should make it a little easier to find the information. Also added note to look below for the definition of the generated methods.
* | | | | | | | | Merge pull request #15280 from sgrif/sg-postgres-defaultsRafael Mendonça França2014-05-233-76/+78
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Move parsing of PG sql strings for defaults out of column
| * | | | | | | | | Move parsing of PG sql strings for defaults out of columnSean Griffin2014-05-233-76/+78
| | |_|/ / / / / / | |/| | | | | | |
* | | | | | | | | Merge pull request #15275 from kuldeepaggarwal/add-testRafael Mendonça França2014-05-232-0/+4
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | add test cases for negative position in Array#from
| * | | | | | | | | add test cases for negative position in Array#fromKuldeep Aggarwal2014-05-232-0/+4
| | |/ / / / / / / | |/| | | | | | |
* | | | | | | | | there is no formatter on the router object, so rmAaron Patterson2014-05-231-1/+0
| | | | | | | | |
* | | | | | | | | Merge pull request #15271 from sgrif/sg-remove-duplicated-setupRafael Mendonça França2014-05-231-4/+3
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Remove duplicated setup in test
| * | | | | | | | | Remove duplicated setup in testSean Griffin2014-05-231-4/+3
| | |/ / / / / / / | |/| | | | | | |
* | | | | | | | | Merge pull request #15273 from DNNX/formatter-refactoring-4Rafael Mendonça França2014-05-231-5/+0
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Remove AD::Journey::Formatter#verify_required_parts!
generated by cgit v1.2.3 (git 2.25.1) at 2024-07-05 06:22:22 +0000