Ticket #562 (closed Bug: fixed)

Opened 5 months ago

Last modified 2 months ago

inconistency in default value of $index param for media::stylesheet()

Reported by: JAAulde Owned by: Shadowhand
Priority: trivial Milestone: 2.2
Component: Modules:Media Version: SVN HEAD
Keywords: Cc:

Description

Media Module - Media helper

with the closure of #526 media::script(), html::script(), html::stylesheet(), and html::link() all set the default value of their $index param to FALSE. but media::stylesheet() still has the $index param set to TRUE.

to keep all of these interfaces consistent, media::stylesheet should also default this param to FALSE

while this is a simple change, it changes the exposed interface so this can't happen until 2.2 this ticket is to remind me to do it

Change History

Changed 2 months ago by Shadowhand

  • owner changed from gregmac to Shadowhand
  • status changed from new to assigned

Actually media::script() should have it's $index param set to TRUE to be consistent.

Fixed in r2987.

Changed 2 months ago by Shadowhand

  • status changed from assigned to closed
  • resolution set to fixed
Note: See TracTickets for help on using tickets.