Deb Lewis
2006-Jun-07 21:03 UTC
[Masterview-devel] branches/preview-template-watcher-refactoring-20060607
Jeff - look this over at some point, I want to pull out a few little utilities that I''m tripping over in various places as I hunt down and fix places where app/views assumptions are lurking. There''s a little readme with some notes plus a couple files I''m tinkering with Needs the new Config stuff before committed, there''s some diffs in constant names and semantics, but you''ll get the general idea ~ Deb -------------- next part -------------- An HTML attachment was scrubbed... URL: http://rubyforge.org/pipermail/masterview-devel/attachments/20060607/ac1125f9/attachment.htm
Jeff Barczewski
2006-Jun-07 23:07 UTC
[Masterview-devel] branches/preview-template-watcher-refactoring-20060607
Yeah I like the idea of cleaning this up and exposing with a block. You are correct this needs to be as simple as possible to allow it to be used from other frameworks. If you are in this code maybe the method name parse_modified_templates sounds better than parse_changed_templates? One question you had in the rails_init.rb regarding whether the server setting belonged in client app settings. I guess we could put this in client settings with a default of what we have here, or depending on how these blocks come out of the refactoring... might just see. Just so you know, the $PROGRAM_NAME check was needed so that the auto parser was run only on script/server, or *cgi launch, and not all the other commands that rails has like script/console script/generate, ... Before I had this the parser was running for everything because the plugin/init.rb gets called before any of those things run. I like what you have started here. On 6/7/06, Deb Lewis <djlewis at acm.org> wrote:> > Jeff - look this over at some point, I want to pull out a few little > utilities that I''m tripping over in various places as I hunt down and fix > places where app/views assumptions are lurking. There''s a little readme > with some notes plus a couple files I''m tinkering with > > Needs the new Config stuff before committed, there''s some diffs in > constant names and semantics, but you''ll get the general idea > > ~ Deb > > _______________________________________________ > Masterview-devel mailing list > Masterview-devel at rubyforge.org > http://rubyforge.org/mailman/listinfo/masterview-devel > >-------------- next part -------------- An HTML attachment was scrubbed... URL: http://rubyforge.org/pipermail/masterview-devel/attachments/20060607/b8d4d99e/attachment.htm