You did nothing wrong: it is the way it is implemented right now. I agree that all that data is overhelming, but you have to conider that - probably - it has been never tried in practice. A better way to do it could be for example:
1. move the DRYML_EDITOR flag (I don't know the reason for such a name) to the config.hobo.xxx namespace, so you can switch it on and off in a more rails 3 way 2. add a few more parameters in the same namespace, so you can decide what you really want to be added in the output 3. optionally deciding a format you might prefer (i.e. html comment, html fragment to be shown in the output) that you can define/override as a regular dryml tag ciao dd On Thu, Apr 14, 2011 at 8:35 AM, Donald R. Ziesig <[email protected]> wrote: > Bryan, > > > Thanks for the tip! > > Unfortunately, when I tried it I got many html comments, as in: > > <!--[DRYML|def|login-page|50|/usr/local/lib/ruby/gems/1.8/gems/hobo-1.3.0.pre31/lib/hobo/rapid/taglibs/rapid_user_pages.dryml[--> > <!--[DRYML|call|simple-page|52|/usr/local/lib/ruby/gems/1.8/gems/hobo-1.3.0.pre31/lib/hobo/rapid/taglibs/rapid_user_pages.dryml[--><!--[DRYML|def|simple-page|4|/usr/local/lib/ruby/gems/1.8/gems/hobo-1.3.0.pre31/lib/hobo/rapid/taglibs/rapid_user_pages.dryml[--> > <!--[DRYML|call|page|5|/usr/local/lib/ruby/gems/1.8/gems/hobo-1.3.0.pre31/lib/hobo/rapid/taglibs/rapid_user_pages.dryml[--><!--[DRYML|def|page|11|app/views/taglibs/application.dryml[--> > <!--[DRYML|call|old-page|12|app/views/taglibs ... > > rendered along with my normal output; so many that the view was pretty much > useless. > > Not knowing of any dryml editors, I used: > > DRYML_EDITOR=abc > export DRYML_EDITOR > > as the enabling definition. (I did hunt through the code for references to > DRYML_EDITOR to see if there was something of use there, but only found a > definition of code_editor which was not used anywhere else, so I had to > presume that any non-blank value of DRYML_EDITOR was sufficient). > > Any idea of what I did wrong? > > Don Z. > > P.S. Your reply below crossed in the mail with my reply to Domizio > outlining a crude but effective mod to the dryml templates. I saw it > yesterday, but had to start the installation of 3000 ft of CAT6 cable in our > new office so had to wait till today to try it. > > On 4/13/2011 1:39 PM, Bryan Larsen wrote: > >> Such a simple solution to a major pain point. So I thought to myself, >> "there must be a reason it hasn't been done.". So I went into Hobo to try >> and hack it in and see what happens. >> >> When I got to the right place, I found this in dryml/template.rb: >> >> def include_source_metadata >> # disabled for now -- we're still getting broken rendering with this >> feature on >> return false >> @include_source_metadata = RAILS_ENV == "development" && >> !ENV['DRYML_EDITOR'].blank? if @include_source_metadata.nil? >> @include_source_metadata >> end >> >> I commented out the "return false" and set the DRYML_EDITOR variable in my >> environment, and it seems to work fine. Give it a try and let me know if >> it breaks anything! >> >> cheers, >> Bryan >> >> P.S. Please don't start new message by replying to an old one unless it's >> actually a reply. Many of us have properly nested email readers, so your >> message becomes hidden in the other thread. >> >> > -- > You received this message because you are subscribed to the Google Groups > "Hobo Users" group. > To post to this group, send email to [email protected]. > To unsubscribe from this group, send email to > [email protected]. > For more options, visit this group at > http://groups.google.com/group/hobousers?hl=en. > > -- You received this message because you are subscribed to the Google Groups "Hobo Users" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/hobousers?hl=en.
