Development

#3127 (sfYaml does parse arrays)

You must first sign up to be able to contribute.

Ticket #3127 (closed defect: fixed)

Opened 2 months ago

Last modified 4 weeks ago

sfYaml does parse arrays

Reported by: dwhittle Assigned to: fabien
Priority: major Milestone: 1.1.0 beta4
Component: configuration Version: 1.1.0 DEV
Keywords: Cc:
Qualification: Unreviewed

Description

The following is no longer parsed correctly:

The value ends up "/static/dw/css/dw-print.css: { media: print }" instead of "/static/dw/css/dw-print.css" => array( media => print ) 

all:
  stylesheets:
    - /static/yui/reset-fonts-grids/reset-fonts-grids.css
    - /static/yui/base/base-min.css
    - /static/yui/assets/skins/sam/skin.css
    - /static/dw/css/dw-global.css
    - /static/dw/css/dw.css
    - /static/dw/css/dw-print.css: { media: print }
    - /static/dw/css/dw-ie.css: { media: all, condition: IE }

Change History

03/16/08 21:20:53 changed by fabien

  • milestone set to 1.1.0.

03/16/08 21:21:22 changed by fabien

in r7922

03/19/08 17:13:14 changed by fabien

  • status changed from new to closed.
  • resolution set to fixed.

04/09/08 01:12:43 changed by dwhittle

  • status changed from closed to reopened.
  • resolution deleted.
  • component changed from other to configuration.

This regression still exists:

For example: 

  javascripts:
    - /static/js/yui/2.5.1/yahoo-dom-event.js
    - /static/js/yui/2.5.1/connection-min.js
    - /static/js/yui/2.5.1/animation-min.js
    - /static/js/yui/2.5.1/autocomplete-min.js
    - /static/js/my-ie.js: { condition: IE }

The last value of the array is still parsed as '/static/js/my-ie.js: { condition: IE }', instead of '/static/js/my-ie.js' => array('condition' => 'ie')

04/10/08 20:48:59 changed by dwhittle

  • priority changed from minor to major.
  • milestone changed from 1.1.0 to 1.1.0 beta4.

04/17/08 00:25:53 changed by fabien

  • status changed from reopened to closed.
  • resolution set to fixed.

in r8489