ExpressionEngine CMS
Open, Free, Amazing

Thread

This is an archived forum and the content is probably no longer relevant, but is provided here for posterity.

The active forums are here.

Image cannot be saved after editing in Image Editor

August 08, 2011 3:29am

Subscribe [7]
  • #16 / Sep 02, 2011 6:59pm

    Robin Sowell

    13255 posts

    Just to keep everyone in the loop- I THINK I have a fix in- whether it’s the permanent solution I’m not sure, as there are a number of ways to attack the problem.

    If you’re running into it on a Windows box- try editing system/codeigniter/system/libraries/image_lib.php around line 209

    else
            {
                if (strpos($this->new_image, '/') === FALSE && strpos($this->new_image, '\\') === FALSE)
                {
                    $this->dest_folder = $this->source_folder;
                    $this->dest_image = $this->new_image;
                    
                }

    I added in the && strpos($this->new_image, ‘\\’) === FALSE bit.  Basically, it was reading the path to the new image as an image name due to the fact it was looking for a forward slash only.

    Again- I need to ponder a bit whether this is where we want to change things.  But it should get you up and running in the short term.  Let me know if you’re still having an issue (could be a lot at play, but worked on bgarrant’s test site issue.)

  • #17 / Sep 02, 2011 8:04pm

    bgarrant

    356 posts

    I will try this Robin and report back. I have about 8 sites using IIS on different servers with this issue.

    Thanks again.

    Bryan

  • #18 / Sep 03, 2011 9:16am

    bgarrant

    356 posts

    Thank you.  This does fix the issue Robin.  Am I safe to make this modification to all 2.2.1 and 2.2.2 sites?

    You may want to correct your post as the file is actually located at:

    system/codeigniter/system/libraries/image_lib.php

    Thanks again.

  • #19 / Sep 04, 2011 10:58pm

    Dan Decker

    7338 posts

    bgarrant,

    Indeed! You are free to make the changed in 2.2.1 and 2.2.2. Thanks for pointing out the correct file path 😊
    Is there anything else we can assist you with?

    Cheers,

  • #20 / Sep 04, 2011 11:06pm

    bgarrant

    356 posts

    All Set with this Dan. You can close.  There is another post I found once to fix the upload image button issue in the control panel. The button does not work in 2.2.2. Do you have a link to the fix by chance? I also have another post I really need help with for an EE Cron issue if you can help. You can send to me privately if easier. Thank you.

  • #21 / Sep 04, 2011 11:17pm

    Dan Decker

    7338 posts

    bgarrant,

    Here is a link to the fix for the File Manager. We try to keep all the communication in the forums, so every one can benefit from the conversation 😊

    Cheers,

  • #22 / Sep 04, 2011 11:22pm

    bgarrant

    356 posts

  • #23 / Sep 04, 2011 11:24pm

    Dan Decker

    7338 posts

    Heh, that’s a link to *this* thread…

    Cheers,

  • #24 / Sep 04, 2011 11:33pm

    bgarrant

    356 posts

  • #25 / Sep 06, 2011 1:14pm

    Kevin Smith

    4784 posts

    I hate to play bad cop here, but we need to keep the discussion focused on the topic at hand. We’re not going to leave anyone in the dark with other threads; we definitely want to help solve those issues, too, but cross-posting your other issues here won’t really speed up the process, and it mucks this thread up for others.

    It appears this is an emerging issue for our users with Windows servers, and we need to make sure this thread remains relevant to anyone else reporting that issue while we confirm that Robin’s fix takes care of things.

    Thanks guys!

  • #26 / Sep 06, 2011 3:27pm

    bgarrant

    356 posts

    Robin’s fix did fix the issue for me Kevin. Thanks again.

  • #27 / Sep 07, 2011 7:25pm

    Dan Decker

    7338 posts

    Marc,

    I’m just pinging you with this post to see where you are on this issue. Have you applied the fix Robin outlined earlier? Since you are the OP on this thread, I wanted to make sure you are all set before resolving it out.

    Cheers,

  • #28 / Sep 07, 2011 7:54pm

    bgarrant

    356 posts

    Please close this ticket Dan. All is working perfect with Robin’s fix.

  • #29 / Sep 07, 2011 9:01pm

    Robin Sowell

    13255 posts

    Good deal, all.  Fix has been committed- for final fix went with

    if (strpos($this->new_image, DIRECTORY_SEPARATOR) === FALSE)

    You should be able to upgrade normally next release and not have to worry about it!

.(JavaScript must be enabled to view this email address)

ExpressionEngine News!

#eecms, #events, #releases