-
Notifications
You must be signed in to change notification settings - Fork 382
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
undefined is not an object (evaluating 'args.length') #129
Comments
Seeing the same thing, using PhantomJS 2.0. Looks like yslow can no longer access the phantomJS arguments. |
Digging in a little, and found that 'phantom.args' is deprecated in v2 (http://phantomjs.org/api/phantom/property/args.html). I tried updating the relevant code to use the newer 'system.args' call and it no longer throws an error, but it also no longer finishes. Bummer. Looks like I'll have to dig a little deeper. |
yep, the |
@lewsid the first argument in system.args is the script name so you need to adjust the arguments, then it will work. |
Thanks for the info @soulgalore! I've applied your fix, and rearranged my arguments, but it still hangs. My command looks like this:
Can you spot anything that I'm doing obviously wrong here? |
@lewsid if you could share your branch, then I can try it out, and then you can make a PR so it's fixed in YSlow? |
@lewsid Did you manage to to get yslow working? I am also looking the fix for this. |
@jpkeisala Unfortunately, no. Sorry to leave you and others hanging without my code for reference. I made some progress but eventually abandoned the effort and just nixed yslow from the project. |
I can make the PR tonight then, np. |
Did this issue fix for now? I'm facing the same issue. I'm using the latest phantomjs(as of
|
@soulgalore Thanks, it works. But do you know how to use yslow for login pages ? I know phantomjs can work as a real browser, but I don't know how to implement. |
removed deprecated args to work with phantomjs 2 marcelduran#129
I'm facing the same issue as well with version 3.1.8 Could anyone help? TypeError: undefined is not an object (evaluating 'args.length') phantomjs://code/yslow.js:5 in global code |
To make it much simpler to follow just do the following. 1.) Open yslow.js file under yslow-phantomjs-3.1.8 The above works for me. |
Problem still exists in yslow-phantomjs-3.1.8 + the above fix is difficult because yslow.js seem obfuscated. Be great if this problem could be fixed. Also, fix show above does not work. Changes the error to...
|
for phantomjs 2.0.0dev
phantomjs yslow.js --help
output is
The text was updated successfully, but these errors were encountered: