Commit ab0ccecca434d315d5939dbe7bedc86fa00285ab
Committed by
Antonio Terceiro
Antonio Terceiro
1 parent
209dd56e
Exists in
master
and in
29 other branches
adding UPGRADE file. This file have the main modifications on system to make the upgrade
Showing
2 changed files
with
27 additions
and
4 deletions
Show diff stats
| @@ -0,0 +1,27 @@ | @@ -0,0 +1,27 @@ | ||
| 1 | +0 - All modified code have the tag UPGRADE on it. | ||
| 2 | + | ||
| 3 | +1 - I Changed acts_as_versioned plugin code to the new one: | ||
| 4 | + | ||
| 5 | + http://github.com/technoweenie/acts_as_versioned/tree/master | ||
| 6 | + | ||
| 7 | +2 - I removed the 'build_person' calls after create the user on UserTest | ||
| 8 | + | ||
| 9 | +3 - The settings_items method generated a lot of bugs. Theses bugs are explained here: | ||
| 10 | + Solution 1: http://casperfabricius.com/site/2008/06/12/serialize-doesnt-play-nice-with-rails-21-partial-updates/ | ||
| 11 | + Solution 2: http://www.kalzumeus.com/2008/06/28/rails-fails-to-update-serialized-columns-on-save/ | ||
| 12 | + | ||
| 13 | +The problem is how rails is working with ActiveRecord. The serialization data was not save on database. | ||
| 14 | + | ||
| 15 | +I add this code | ||
| 16 | + | ||
| 17 | + send(self.class.settings_field.to_s + '_will_change!') | ||
| 18 | + | ||
| 19 | +on lib/acts_as_having_settings | ||
| 20 | + | ||
| 21 | +I used the solution 1. | ||
| 22 | + | ||
| 23 | +4 - acts_as_searchable -> find_by_contents | ||
| 24 | + | ||
| 25 | +I solved an incompatibility between paginate and geokit with an work around like this the one explained on the link: | ||
| 26 | + | ||
| 27 | + http://groups.google.com/group/will_paginate/browse_thread/thread/1bbb1f0c5810f1c1 |
vendor/plugins/will_paginate/.gitignore