Skip to content

ERROR: This is not correct. Disregard this. added code to remove empty space that was producing an error: Cropping e... #103

Open
wants to merge 4 commits into from

2 participants

@hguillermo

Hi,

I was receiving this error:

TypeError: Cannot read property 'geometry' of undefined
at null.callback (/home/patison/node_modules/imagemagick/imagemagick.js:155:26)
at Accumulator.finish (/home/patison/node_modules/imagemagick/imagemagick.js:60:55)
at ChildProcess. (/home/patison/node_modules/imagemagick/imagemagick.js:85:11)
at ChildProcess.EventEmitter.emit (events.js:91:17)
at Process._handle.onexit (child_process.js:674:10)

The problem was the spaces in front of the output file that the method was parsing. Fixed now.

@hguillermo hguillermo added code to remove empty space that was producing an error: Croppin…
…g error (TypeError: Cannot read property geometry of undefined)
e871530
@h-evers
h-evers commented Jul 22, 2013

+1 for this, needed for me too (Windows, ImageMagick 6.8.6-3 2013-06-25 Q16)

@hguillermo

Hey,

It seems like the maintainers are not really merging pull requests. Maybe I need to contact them to see if I can help.

@h-evers
h-evers commented Jul 22, 2013

I added it by hand locally, but "crop" still doesn't work - needed property "geometry" is missing in "identify" data... So i'm going to adapt it for my need.

@hguillermo

Huh! I had the same problem with crop but now it works. Check that method and print the output. I am sure that value is coming unless your ImageMagick is too old.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.