cms_manageable.rb tags accessor

[Emerson]

def tags(force_reload = false)
  self.render if force_reload
  @tags ||= []
end

The force reload can cause issues when working with initialized (but unsaved) blocks. It's called by the block.rb#tag method.

Might be a non-issue, but we should investigate.

[eirikurn]

Not sure if it's related but when I do cms_block_content f.ex. in a navigation (maybe not the bestest practice), I see 7 db queries for every page. It seems to load each page, layout, parent layout, blocks and even all linked files, even if I'm just asking for one "simple" string block by identifier.

cms_block_content calls block.rb#tag, which runs tags(:reload).

[GBH]

@eirikurn It's somewhat related. When you use cms_block_content it runs though the tag system to figure out what exactly needs to be rendered out. If you want to grab block content quickly just go through AR: @cms_page.blocks.find_by(:identifier => 'whatever').content

[GBH]

As of 2.0 Tag parsing system is completely redone. Page does not have tags method anymore.