nlsclientscript Follow @yiiframework

Well, i've just have seen that http://code.google.com/p/cssmin/ is able to merge the @import-ed css files into the parent one by its documentation. That's indeed a feature what makes me consider to use it in a later version some way. Thanks for the tip...

Hi, i'm not sure the problem relates to the minification. When the extension merges the files, it publishes the merged one into the /assets dir. That way, if you use relative url in your @import (like "dir/xy.css" or "../dir/xy.css"), that will usually break.

Have you tried to use url like "/dir/xy.css" in the @import rule (relative to the server root) or perhaps full absolute url?

About cssmin, i try to keep my extension lightweight, without merging 3rd parties into being 10x large than the extension itself, unless it is really necessary.

Thanks, nice extension, but import function of css not work after minify. May be you should use full css minifier (e.g http://code.google.com/p/cssmin/) not simple css minifier?

yes i'm aware, will fix soon. Till then set that param explicitly in config.php

$_SERVER['REQUEST_SCHEME'] is not always available, and will throw undefined index:

if (!$this->serverBaseUrl)
            $this->serverBaseUrl = $_SERVER['REQUEST_SCHEME'] . '://' . $_SERVER['HTTP_HOST'];

Omitting it is safe in most browsers, but I haven't checked how it affects other parts of your code:

if (!$this->serverBaseUrl)
            $this->serverBaseUrl = '//' . $_SERVER['HTTP_HOST'];

Hello nlac, Thanks so much for such work, was totally a savior for me.

I'm reporting that there's a conflict between nlsclientscript & Minscript extension http://www.yiiframework.com/extension/minscript/

The js file is loaded again & duplicated after the first ajax request.. thats because the normUrl function of nlsclientscript takes the minscript url in the form of:

Http: //localhost/soedu/min/serve?g=7668aa4b832f39e552dd77ac5e156c5c&lm=1353462851&_=1355498134802

& returns

Http: //localhost/soedu/min/serve?g=7668aa4b832f39e552dd77ac5e156c5c&lm=1353462851&

Notice the '&' at the end

The suggested fix would be to modify normUrl function, replacing

return url.replace(/\?*(_=\d+)?$/g,"");

With

return url.replace(/(\?|&)*(_=\d+)?$/g,"");

Even with the extension when using AJAX, the Bootstrap CSS files are reloaded the page.

Hi

I just tried version 5.0 after previously using v3.6.

I use it as in the example: $("#cont").html($.ajaxSettings.dataFilter(data.content)); . Apparently this does not work anymore ad 'dataFilter' is unknown in Chrome (for example) when using 5.0. It appears that 'dataFilter' is defined only when using 'IE'.

Using version 5 seems to work as expected in IE7/8 and Chrome. I upgraded after IE7/8-issues in version 3.3.

Only thing I would like to see is proper DocBlock commenting on variables and functions in your extension.

And actually, maybe a better name because "NLSClientScript" doesn't say anthing about what it does, but something like "AjaxCompatibleClientScript" or maybe just "EClientScript" or "XClientScript" is better.

You are right. My 'static' pages register a small js script. But how you said they don't need jquery.

I register now jquery for all my sites in my main layout view.

I use the google lib and I think it is no problem for performance.

I guess that is not a static page where the issue occurs. On the pages where no script file registered, the component renders nothing. On your "static page" some script is registered requires no jquery, that's when the issue occurs.

So yes, for now all pages need jquery registered since the component uses it. I'll fix in the next release to register jquery automatically in every case.

I have install version 5.0.

It works on all my pages where jquery is included.

But on my static pages (e.g. impressum page) where I need no jquery there I get the error 'jquery undefined'.

What can I do. Must I include on all my pages jquery or can I do this with exclude pattern ?

test results about usages of the latest version are welcome

IE7 has a problem with the "src" attribute of the script tag, as can be seen in http://bugs.jquery.com/ticket/11404

So the update of the grid returns the alert "Parser error".

The solution to the problem would be to change the script to:

(...)
selector : "script:not([src=\"\"]),link[rel=\"stylesheet\"]",
(...)

is #7098 fixed in 3.6 version? I am still facing the problem in filtering in the cgridview

I have problems. When the extension is enabled the HTML content is being duplicated in ajax requests.

Detail: the problem does not occur in IE, only in Firefox and Chrome. And only occurs when the return of Ajax is the complete page, including the HTML tag.

The javascript code behaves completely different in IE vs Firefox and Chrome.

The developer needs to fix the default behavior on all browsers.

A workaround for Firefox and Chrome (but certainly not the ultimate solution):

(...)
firstTagRg : /<([a-z1-6]+)[^>]*>/i,
(...)
holders : {
    "default" : $(document.createElement("div")),
    "tr" : $(document.createElement("tbody")),
    "html" : $(document.createElement("html"))
},

hoplayann thanks for the report, i'll check and will release a new version in a few days, there's also an other issue needs to be fixed.

Hello,

I am using version 3.6 of your extension in my Yii project.

It seems that the selection of script tags in the HTML source fails for some reason in Internet Explorer 7. The looping through the SCRIPT or LINK elements is the place where we have now added an extra test case, to be sure that there is at least a not empty SRC or HREF attribute defined. (see added code below)

//fetching scripts from the DOM
                    for(i=0,res=$(document).find($._nlsc.selector); i<res.length; i++) {
                        tmp = res[i];
 
                        // CODE ADDED:
                        // ie7 sometimes gets accidental script tags and selector returns undefined objects
                        if (tmp.src == "" && typeof(tmp.href) === "undefined") {
                            continue;
                        }
 
                        ind = tmp.src ? tmp.src : tmp.href;
                        ind = ind.replace($._nlsc.urlGarbageRg,"");
                        if (tmp.href && tmp.parentNode!=$._nlsc.head)
                            $._nlsc.head.appendChild(tmp);
                        $._nlsc.resMap[ind] = 1;
                    }

And I had to add the code again in the other loop :

// iterate over new scripts and remove if source is already in DOM:
                for(i=0,res=holder.find($._nlsc.selector); i<res.length; i++) {
                    tmp = res[i];
 
                    // CODE ADDED:
                    // ie7 sometimes gets accidental script tags and selector returns undefined objects
                    if (tmp.src == "" && typeof(tmp.href) === "undefined") {
                        continue;
                    }
 
                    ind = tmp.src ? tmp.src : tmp.href;
                    ind = ind.replace($._nlsc.urlGarbageRg,"");
                    if (resMap[ind])
                        $(tmp).remove();
                    else {
                        if (tmp.href)
                            $._nlsc.head.appendChild(tmp);
                        else
                            tmp.removeAttribute("defer");
                        resMap[ind] = 1;
                    }
                }

Is it possible that the jquery selector is not precise enough ?

selector : "script[src],link[rel=\"stylesheet\"]",

Please let me know if there would be a fix for that issue in your next revision.

Yann

fixed in 3.6

Here is returning an error warning that "head" is not defined.

This piece of code:

// iterate over new scripts and remove if source is already in DOM:
for(i=0,res=holder.find($._nlsc.selector); i<res.length; i++) {
    tmp = res[i];
    ind = tmp.src ? tmp.src : tmp.href;
    ind = ind.replace($._nlsc.urlGarbageRg,"");
    if (resMap[ind])
        $(tmp).remove();
    else {
        if (tmp.href)
            head.appendChild(tmp);
        else
            tmp.removeAttribute("defer");
        resMap[ind] = 1;
    }
}

should be?

// iterate over new scripts and remove if source is already in DOM:
for(i=0,res=holder.find($._nlsc.selector); i<res.length; i++) {
    tmp = res[i];
    ind = tmp.src ? tmp.src : tmp.href;
    ind = ind.replace($._nlsc.urlGarbageRg,"");
    if (resMap[ind])
        $(tmp).remove();
    else {
        if (tmp.href)
            $._nlsc.head.appendChild(tmp);
        else
            tmp.removeAttribute("defer");
        resMap[ind] = 1;
    }
}