Skip to content
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

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
base: master
Choose a base branch
from

Conversation

hguillermo
Copy link

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.

…g error (TypeError: Cannot read property geometry of undefined)
@h-evers
Copy link

h-evers commented Jul 22, 2013

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

@hguillermo
Copy link
Author

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
Copy link

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
Copy link
Author

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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants