aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | Merge pull request #5277 from rafaelfranca/fix-buildMichael Koziarski2012-03-041-0/+1
|\ \ \ \ \ | | | | | | | | | | | | Now all the models need to explicitly declare the accessible attributes
| * | | | | Now all the models need to explicitly declare the accessible attributesRafael Mendonça França2012-03-041-0/+1
|/ / / / /
* | | | | Merge pull request #5275 from rafaelfranca/masterJon Leighton2012-03-043-2/+11
|\ \ \ \ \ | | | | | | | | | | | | Only add the whitelist_attributes option if ActiveRecord is present
| * | | | | Only add the whitelist_attributes option if ActiveRecord is presentRafael Mendonça França2012-03-043-2/+11
|/ / / / /
* | | | | Whitelist all attribute assignment by default.Michael Koziarski2012-03-054-1/+20
| | | | | | | | | | | | | | | | | | | | Change the default for newly generated applications to whitelist all attribute assignment. Also update the generated model classes so users are reminded of the importance of attr_accessible.
* | | | | Fix tests, table_name= needs connection now, so just stub table_namePiotr Sarnacki2012-03-041-1/+1
| | | | |
* | | | | Merge pull request #5271 from carlosantoniodasilva/ar-fix-buildJosé Valim2012-03-042-2/+2
|\ \ \ \ \ | |/ / / / |/| | | | Fix message assertions for quoting database name in "show tables" for mysql
| * | | | Fix message assertions for quoting database name in "show tables" for mysqlCarlos Antonio da Silva2012-03-042-2/+2
|/ / / /
* | | | Merge pull request #5268 from carlosantoniodasilva/ar-new-record-warningJon Leighton2012-03-043-1/+10
|\ \ \ \ | | | | | | | | | | Fix ActiveRecord warning in tests with Marshal
| * | | | Initialize @stale_state to nil in associationCarlos Antonio da Silva2012-03-043-6/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This apparently fix the warning related to @new_record variable not being initialized in AR's test suit, when an association is built and the object is marshalled/loaded. See these tests in AR's base_test.rb: test_marshalling_with_associations test_marshalling_new_record_round_trip_with_associations Closes #3720.
| * | | | Added test case for new_record round trip with associations problemkennyj2012-03-041-0/+14
| | | | |
* | | | | remove trollJon Leighton2012-03-041-3/+0
| | | | |
* | | | | tidy up formattingJon Leighton2012-03-041-10/+11
| | | | |
* | | | | Merge pull request #5267 from kennyj/fix_4674Jon Leighton2012-03-042-5/+38
|\ \ \ \ \ | | | | | | | | | | | | Fix GH #4674. Reset column information and sequence name when setting table_name.
| * | | | | Don't clear sequence name when we explicitly assign it.kennyj2012-03-042-2/+15
| | | | | |
| * | | | | Fix GH #4674. Reset column information and sequence name when setting ↵kennyj2012-03-042-4/+24
| | | | | | | | | | | | | | | | | | | | | | | | table_name.
* | | | | | wow how come I commit in master? O_ohoma2012-03-041-0/+3
| | | | | |
* | | | | | Merge pull request #5269 from ↵Piotr Sarnacki2012-03-041-1/+0
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | castlerock/remove_unwanted_require_karnel_reporting AS/kernel/reporting no where used in ts_isolated
| * | | | | | AS/kernel/reporting no where used in ts_isolatedVishnu Atrai2012-03-041-1/+0
| |/ / / / /
* | | | | | Merge pull request #5266 from bcardarella/root_with_string_optionsPiotr Sarnacki2012-03-042-0/+14
|\ \ \ \ \ \ | | | | | | | | | | | | | | Route root helper shortcut
| * | | | | | Route root helper shortcutBrian Cardarella2012-03-032-0/+14
| | |_|/ / / | |/| | | | | | | | | | | | | | | | Allow the root route helper to accept just a string
* | | | | | Merge pull request #5270 from kennyj/fix_3163Piotr Sarnacki2012-03-043-1/+22
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Fix GH #3163. Should quote database on mysql/mysql2.
| * | | | | Fix GH #3163. Should quote database on mysql/mysql2.kennyj2012-03-053-1/+22
|/ / / / /
* | | | | Fix rake test_sqlite3_memJon Leighton2012-03-041-7/+9
| | | | |
* | | | | Remove unusused variablesJon Leighton2012-03-043-14/+9
|/ / / /
* | | | [engines guide] Add more concrete example of conflicts when not using ↵Piotr Sarnacki2012-03-031-1/+1
| | | | | | | | | | | | | | | | isolate_namespace
* | | | Handle files from ActionDispatch::Static with Rack::Sendfile (fixes #5225)Piotr Sarnacki2012-03-033-5/+18
|/ / / | | | | | | | | | | | | | | | | | | | | | This makes rails behave properly when you serve static assets and you have X-Sendfile headers enabled. Nevertheless in most cases you should not rely on that and serve static assets with a webserver like Apache or Nginx (as you already have it in place anyway if you use X-Sendfile)
* | | Merge pull request #5255 from carlosantoniodasilva/active-record-reviewPiotr Sarnacki2012-03-038-68/+77
|\ \ \ | | | | | | | | Refactor and cleanup in some ActiveRecord modules
| * | | Refactor and cleanup in some ActiveRecord modulesCarlos Antonio da Silva2012-03-038-68/+77
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Avoid double hash lookups in AR::Reflection when reflecting associations/aggregations * Minor cleanups: use elsif, do..end, if..else instead of unless..else * Simplify DynamicMatchers#respond_to? * Use "where" instead of scoped with conditions hash * Extract `scoped_by` method pattern regexp to constant * Extract noisy class_eval from method_missing in dynamic matchers * Extract readonly check, avoid calling column#to_s twice in persistence * Refactor predicate builder, remove some variables
* | | | Merge pull request #5246 from lest/patch-2José Valim2012-03-031-1/+1
|\ \ \ \ | | | | | | | | | | simplify namespace assignment in fields_for
| * | | | simplify namespace assignment in fields_forSergey Nartimov2012-03-021-1/+1
| | | | |
* | | | | Merge pull request #5260 from castlerock/remove_ordered_hash_usagesSantiago Pastorino2012-03-035-6/+6
|\ \ \ \ \ | | | | | | | | | | | | remove usage of AS::OrderedHash from other places
| * | | | | remove usages of AS::OrderedHashVishnu Atrai2012-03-035-6/+6
|/ / / / /
* | | | | Merge pull request #4984 from raghunadhd/change_orderhashVijay Dev2012-03-033-3/+3
|\ \ \ \ \ | |_|_|_|/ |/| | | | Refactored code
| * | | | Refactored codeRaghunadh2012-02-103-3/+3
| | | | |
* | | | | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-03-036-19/+65
|\ \ \ \ \
| * | | | | copy edits [ci skip]Vijay Dev2012-03-031-1/+1
| | | | | |
| * | | | | Made documentation of config.assets.precompile slightly more explicit.Guy Bolton King2012-03-021-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Having read the previous version of the documentation, I was surprised when a wildcarded CSS pattern in precompile picked up a whole load of SCSS files! I've amended lines 426 onwards to document the behaviour.
| * | | | | Removed max-stale from the setting the cache-headers in the response as ↵Anuj Dutta2012-03-021-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | max-stale is a cache request header.
| * | | | | Add documentation for IndexDefinition.Waseem Ahmad2012-03-021-0/+3
| | | | | |
| * | | | | Improve header hierarchy.Joost Baaij2012-02-291-1/+3
| | | | | |
| * | | | | Include a comment as to why ETag header should be removed from the response.Joost Baaij2012-02-291-0/+1
| | | | | |
| * | | | | Changed asset_host config to more specificRahul P. Chaudhari2012-02-291-2/+2
| | | | | |
| * | | | | Some cleanup on the upgrading guideMike Gunderloy2012-02-281-14/+40
| | | | | |
| * | | | | Merge pull request #85 from whilefalse/rails_32_engine_upgradeVijay Dev2012-02-281-0/+12
| |\ \ \ \ \ | | | | | | | | | | | | | | Added guide to updating engines built in previous versions of Rails
| | * | | | | Added guide to updating engines built in previous versions of RailsSteven Anderson2012-02-241-0/+12
| | | | | | |
* | | | | | | Merge pull request #5258 from guilleiguaran/amo-modelJon Leighton2012-03-031-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Use public_send instead of send in ActiveModel::Model initialize
| * | | | | | | Use public_send instead of send in ActiveModel::Model initializeGuillermo Iguaran2012-03-031-1/+1
|/ / / / / / /
* | | | | | | Merge pull request #5256 from carlosantoniodasilva/active-model-modelJosé Valim2012-03-033-1/+81
|\ \ \ \ \ \ \ | |_|_|_|_|/ / |/| | | | | | Add docs with usage examples for ActiveModel::Model
| * | | | | | Add docs with usage examples for ActiveModel::ModelCarlos Antonio da Silva2012-03-033-1/+81
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | Also add test to ensure basic model does not explode when initialized with nil.