On 02/01/2012 10:59 AM, Jan Provaznik wrote:
On 01/31/2012 04:26 PM, ifarkas@redhat.com wrote:
From: Imre Farkasifarkas@redhat.com
https://bugzilla.redhat.com/show_bug.cgi?id=781579
src/app/controllers/deployables_controller.rb | 8 ++++++ src/app/views/deployables/show.html.haml | 33 +++++++++++++++++++++++++ src/public/javascripts/backbone/models.js | 10 +++++++ src/public/javascripts/backbone/routers.js | 19 ++++++++++++++ src/public/javascripts/backbone/views.js | 13 +++++++++- 5 files changed, 82 insertions(+), 1 deletions(-)
ACK
OT note: I'm little bit scared of how much code is duplicated for html/backbone views. Would be great to do deeper investigation how to optimize this (in post 1.0).
Jan
+1 to this.
On at least two separate occasions I fixed a bug, only to realize that the problem was "un-fixed" on refresh, since the auto-updated tables are repopulated via a completely different code path than the original population.
Also, the refresh code is organized in such a different way than the controller code that it's hard to connect it (mentally) with the related controller code to confirm that things are really doing the same thing.
Scott