Issues

ZF-10327: ZendX_Application_Resource_Jquery does not allow CDN_SSL

Issue Type: Patch Created: 2010-08-17T10:25:30.000+0000 Last Updated: 2011-07-05T15:08:35.000+0000 Status: Resolved Fix version(s): - 1.11.8 (07/Jul/11)

Reporter: Steven Hadfield (sthadfield) Assignee: Kim Blomqvist (kblomqvist) Tags: - ZendX_JQuery

Related issues: Attachments: - Container.php.patch

Description

The current application resource for Jquery does not allow cdn_ssl to be enabled. This is explicitly disabled by the array_merge of the options with array('cdn_ssl' => false)

Here's my sample portion of the ini file

<pre class="highlight">
resources.Jquery.version = 1.4.2
resources.Jquery.ui_enable = true
resources.Jquery.ui_version = 1.8.4
resources.Jquery.cdn_ssl = true
resources.Jquery.stylesheet = "https://ajax.googleapis.com/ajax/libs/jqueryui/1.8.4/themes/ui-lightness/jquery-ui.css"

But the configuration for 'cdn_ssl' is ignored because it is merged with an array where it is set to false

<pre class="highlight">
    protected function _parseOptions(array $options)
    {
        $options = array_merge($options, array('cdn_ssl' => false));

If there's a reason for it being done, I can't find any documentation for it.

The fix would be just to remove the array_merge. The workaround is to set ``` but this defeats the purpose of having that configurability in the resource. I need cdn_ssl to prevent mixed-mode when other resources are made available over SSL.

Comments

Posted by Kim Blomqvist (kblomqvist) on 2011-04-23T13:09:10.000+0000

Patch attached

Posted by Matthew Weier O'Phinney (matthew) on 2011-07-05T15:08:35.000+0000

Patch applied to trunk and 1.11 release branch.

Have you found an issue?

See the Overview section for more details.

Copyright

© 2006-2016 by Zend, a Rogue Wave Company. Made with by awesome contributors.

This website is built using zend-expressive and it runs on PHP 7.

Contacts