TypeError: Cannot call method 'clone' of undefined

Hi there,

Have encountered this errror, which is eliminated when I use --skip-advanced.

Any ideas on what I should be looking for?

/usr/local/lib/node_modules/clean-css/lib/properties/shorthand-compactor.js:86
var c = componentsSoFar[prop].lastShorthand.components[i].clone(isIm
^
TypeError: Cannot call method 'clone' of undefined
at compactSoFar (/usr/local/lib/node_modules/clean-css/lib/properties/shorthand-compactor.js:86:69)
at compactAllSoFar (/usr/local/lib/node_modules/clean-css/lib/properties/shorthand-compactor.js:165:18)
at Object.module.exports.compactShorthands (/usr/local/lib/node_modules/clean-css/lib/properties/shorthand-compactor.js:217:5)
at Optimizer.compact (/usr/local/lib/node_modules/clean-css/lib/properties/optimizer.js:253:33)
at Object.Optimizer.process (/usr/local/lib/node_modules/clean-css/lib/properties/optimizer.js:270:18)
at Optimizer.optimize (/usr/local/lib/node_modules/clean-css/lib/selectors/optimizer.js:289:40)
at Object.Optimizer.process (/usr/local/lib/node_modules/clean-css/lib/selectors/optimizer.js:337:7)
at Object.optimizeSelectors as 0
at replace (/usr/local/lib/node_modules/clean-css/lib/clean.js:103:19)
at CleanCSS.minify (/usr/local/lib/node_modules/clean-css/lib/clean.js:354:5)

[GoalSmashers]

@geoclicks could you please share CSS you are trying to minify?

[joe-was-here]

I'm getting the same issue. It's a fair amount of CSS, 8000 lines or so.

http://www.speedyshare.com/ZQRCX/custom.dom.css

And here is the error

/Users/joe/development/next/node_modules/gulp-minify-css/node_modules/clean-css/lib/properties/shorthand-compactor.js:86
var c = componentsSoFar[prop].lastShorthand.components[i].clone(isIm
^
TypeError: Cannot call method 'clone' of undefined
at compactSoFar (/Users/joe/development/next/node_modules/gulp-minify-css/node_modules/clean-css/lib/properties/shorthand-compactor.js:86:69)
at compactAllSoFar (/Users/joe/development/next/node_modules/gulp-minify-css/node_modules/clean-css/lib/properties/shorthand-compactor.js:165:18)
at Object.module.exports.compactShorthands (/Users/joe/development/next/node_modules/gulp-minify-css/node_modules/clean-css/lib/properties/shorthand-compactor.js:217:5)
at Optimizer.compact (/Users/joe/development/next/node_modules/gulp-minify-css/node_modules/clean-css/lib/properties/optimizer.js:253:33)
at Object.Optimizer.process (/Users/joe/development/next/node_modules/gulp-minify-css/node_modules/clean-css/lib/properties/optimizer.js:270:18)
at Optimizer.optimize (/Users/joe/development/next/node_modules/gulp-minify-css/node_modules/clean-css/lib/selectors/optimizer.js:289:40)
at Optimizer.optimize (/Users/joe/development/next/node_modules/gulp-minify-css/node_modules/clean-css/lib/selectors/optimizer.js:291:9)
at Object.Optimizer.process (/Users/joe/development/next/node_modules/gulp-minify-css/node_modules/clean-css/lib/selectors/optimizer.js:337:7)
at Object.optimizeSelectors as 0
at replace (/Users/joe/development/next/node_modules/gulp-minify-css/node_modules/clean-css/lib/clean.js:103:19)
at CleanCSS.minify (/Users/joe/development/next/node_modules/gulp-minify-css/node_modules/clean-css/lib/clean.js:354:5)

[GoalSmashers]

@jmadden85 thanks for sharing the CSS - it's due to a padding: ; property at line 7216. We should get rid of such properties automatically.

[joe-was-here]

I saw that right as I was uploading, and wasn't sure if that was the culprit. Thanks very much.

The css code is honestly very nasty. Any chance we can put in a debug
statement somewhere to print out what the offending line is?

Thanks for the quick response.
On 15 Aug 2014 04:31, "Goal Smashers Dev Team" notifications@github.com
wrote:

@geoclicks https://github.com/geoclicks could you please share CSS you
are trying to minify?

—
Reply to this email directly or view it on GitHub
#339 (comment)
.

[GoalSmashers]

Definitely, stay tuned for an update during the weekend.

Seems to be the same issue.

.body_wrapper {
padding: 0;
background: transparent none;
-moz-border-radius: ;
-webkit-border-radius: ;
border-radius: ;
}

@jmadden85 Quick FYI. We have worked around this by adding doing the following on the css, before we pass it to clean-css.

sed -i '/: ;/d' nasty.css

[plusplusben]

It would be great if there was an option to have clean css remove lines like this. I'm autogenerating some CSS with LESS and unfortunately some of our properties end up like this.

[jakubpawlowicz]

Fix is ready & will be released soon!

[jakubpawlowicz]

It's out in 2.2.14.