Ticket #573 (closed Bug: wontfix)

Opened 7 months ago

Last modified 2 months ago

Media Module Caching

Reported by: allain Owned by: gregmac
Priority: major Milestone:
Component: Modules:Media Version: SVN HEAD
Keywords: caching Cc:

Description

When caching is enabled only the contents of the last CSS file makes it into the cache.

Line 81 is currently

($this->use_cache) AND $this->cache->set('media.css.'.$querystr, $output, array('media'), $this->cache_lifetime);

but I think it should read:

($this->use_cache) AND $this->cache->set('media.css.'.$querystr, $output, array('media'), $this->cache_lifetime);

Change History

in reply to: ↑ description   Changed 7 months ago by allain

Replying to allain: Sorry, I'm half asleep the original has $data instead of $output

When caching is enabled only the contents of the last CSS file makes it into the cache. Line 81 is currently {{{ ($this->use_cache) AND $this->cache->set('media.css.'.$querystr, $output, array('media'), $this->cache_lifetime); }}} but I think it should read: {{{ ($this->use_cache) AND $this->cache->set('media.css.'.$querystr, $output, array('media'), $this->cache_lifetime); }}}

  Changed 6 months ago by Shadowhand

  • milestone changed from 2.1.2 to 2.1.3

  Changed 4 months ago by caktux

Same thing at line 115 for js()

  Changed 4 months ago by anonymous

  • milestone deleted

Milestone 2.1.3 deleted

  Changed 2 months ago by Shadowhand

  • status changed from new to closed
  • resolution set to wontfix

Media is no longer a supported module, see KohanaModules.

Note: See TracTickets for help on using tickets.