Zend Framework

HeadStyle view helper: allow multiple media descriptors

Details

  • Type: Improvement Improvement
  • Status: Resolved Resolved
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.5.2
  • Fix Version/s: 1.7.2
  • Component/s: Zend_View
  • Labels:
    None

Description

There is a check on the given media attribute to see if it is valid. (file HeadStyle.php, on line 317 - 321)
The check only verifies single values. It should need to support multiple values.

eg.
//Works
$this->headStyle()->captureStart("APPEND", array("media" => "screen"));
//Result
<style type="text/css" media="screen">

//Doesn't work
$this->headStyle()->captureStart("APPEND", array("media" => "screen, projection"));
$this->headStyle()->captureStart("APPEND", array("media" => array("screen", "projection")));

//Expected result
<style type="text/css" media="screen, projection">

Reference media descriptors

Activity

Hide
Matthew Weier O'Phinney added a comment -

Assigning to Jon Whitcraft.

Show
Matthew Weier O'Phinney added a comment - Assigning to Jon Whitcraft.
Hide
Jon Whitcraft added a comment -

This was fixed with r13182

Show
Jon Whitcraft added a comment - This was fixed with r13182

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: