Announcement: New plug-in SuperSizer

Have a question or a suggestion about a 3rd party addon module or plugin?
Let us know here.
Locked
darconny
Forum Members
Forum Members
Posts: 104
Joined: Thu Nov 22, 2007 12:43 pm

Re: Announcement: New plug-in SuperSizer

Post by darconny »

Well, I didn't ask for it, I am very happy with solution that works  ;D

So, when I put new plugin file I get this:

Warning:
getimagesize(http://localhost/cms-1.6.6/uploads/cgblog/id35/img 105.jpg) [function.getimagesize]: failed to open stream: HTTP request failed! HTTP/1.1 404 Not Found in C:\xampp\htdocs\cms-1.6.6\plugins\function.supersizer.php on line 41

Warning: Division by zero in C:\xampp\htdocs\cms-1.6.6\plugins\function.supersizer.php on line 252

Nothing changed in my setup, without adding new parameters.

cheers
Darko
JeremyBASS

Re: Announcement: New plug-in SuperSizer

Post by JeremyBASS »

:D well I had a few requests for the smaple to speed up bysides in here ... but I think I solved the issue you had... in last versions I had


$this->strOriginalImagePath = str_replace(" ", "%20", $strPath);

thou I try to fix it for one of  my tests that failed... it fixed it for me but looks like it failed for you...

now I did

$this->strOriginalImagePath = htmlentities($strPath);

and it fixes my test... and should be good for you too...

tested here

http://www.digitalbarn.tv/testpage.html

but the last file had a no no in the name.. Camp'09 011.jpg which is why the attempt at the fix...

the other issues where on the filters of png and gifs...



All should be good now... :D

let me know if any one has any other issues...

Cheers
Jeremy
Attachments

[The extension txt has been deactivated and can no longer be displayed.]

darconny
Forum Members
Forum Members
Posts: 104
Joined: Thu Nov 22, 2007 12:43 pm

Re: Announcement: New plug-in SuperSizer

Post by darconny »

Ok, here are some other issues  :D

Warning:
getimagesize(http://localhost/cms-1.6.6/uploads/cgblog/id35/img 105.jpg) [function.getimagesize]: failed to open stream: HTTP request failed! HTTP/1.1 404 Not Found in C:\xampp\htdocs\cms-1.6.6\plugins\function.supersizer.php on line 45

Warning: Division by zero in C:\xampp\htdocs\cms-1.6.6\plugins\function.supersizer.php on line 256

Darko
JeremyBASS

Re: Announcement: New plug-in SuperSizer

Post by JeremyBASS »

what the heck... it's that $this->strOriginalImagePath = htmlentities($strPath);  line... hmm... it should have worked for you...

see getimagesize(http://localhost/cms-1.6.6/uploads/cgblog/id35/img 105.jpg) should have been

getimagesize(http://localhost/cms-1.6.6/uploads/cgbl ... %20105.jpg) which would/should have worked...

I'll have to read up as nothing seems to work with the spaces and what not... and you cleared the image cache first?

Jeremy
darconny
Forum Members
Forum Members
Posts: 104
Joined: Thu Nov 22, 2007 12:43 pm

Re: Announcement: New plug-in SuperSizer

Post by darconny »

Hmm, what image cache I have to clear? Maybe I missed something here..
JeremyBASS

Re: Announcement: New plug-in SuperSizer

Post by JeremyBASS »

oh the SuperSizerTmp folder... but that is a long shot... and shouldn't pay in to this as I think of it... Just stumped on what would be the 100% thing to do... it work on all my test beds... hmmm
fredp
Forum Members
Forum Members
Posts: 218
Joined: Sun Jul 27, 2008 1:36 am
Location: USA

Re: Announcement: New plug-in SuperSizer

Post by fredp »

JeremyBASS wrote: • Sample- (optional) Default:true - This choose a low quality resize which can be much faster in the processing time.  Remember the image is only resized once and then cached so this is only needed for sites with ultra high turnover on the resized images.
...
Hi Jeremy,

I just tried the last posted version. It works for me.  I tried it with no Sample arg, with Sample=true, and with Sample=false.  All worked as expected -- Sample=false resulted in much faster cache reload times.  :)

Also, I suggest one minor change to the help text, for clarity:
• Sample- (optional) Default:true - Set false to choose a low quality resize which can be much faster in the processing time.  Remember the image is only resized once and then cached so this is only needed for sites with ultra high turnover on the resized images.

Hope this helps,
Fred
Nearly all men can stand adversity, but if you want to test a man's character, give him power.
- Abraham Lincoln
darconny
Forum Members
Forum Members
Posts: 104
Joined: Thu Nov 22, 2007 12:43 pm

Re: Announcement: New plug-in SuperSizer

Post by darconny »

JeremyBASS wrote: oh the SuperSizerTmp folder... but that is a long shot... and shouldn't pay in to this as I think of it... Just stumped on what would be the 100% thing to do... it work on all my test beds... hmmm
I just tried this.. deleted all images in SuperSizerTmp folder.. the same result, I get these warnings.

cheers
Darko
JeremyBASS

Re: Announcement: New plug-in SuperSizer

Post by JeremyBASS »

hmm.. figures... my test bed works... works for fredp.... hmm...  may-be htmlspecialchars() would be the ticket... I try that in just a bit...

If you two could give me the orgian image paths that would show me what up... why the difference... thank you two for testing here...
fredp
Forum Members
Forum Members
Posts: 218
Joined: Sun Jul 27, 2008 1:36 am
Location: USA

Re: Announcement: New plug-in SuperSizer

Post by fredp »

JeremyBASS wrote: If you two could give me the orgian image paths that would show me what up...
Done -- Sent by PM.

Fred
Nearly all men can stand adversity, but if you want to test a man's character, give him power.
- Abraham Lincoln
JeremyBASS

Re: Announcement: New plug-in SuperSizer

Post by JeremyBASS »

Ok.. So I bet if you were to take one of the images and folders and put a space in them you'll see the folder... it's the spaces and ' that seem to plague this... I'll have to see  darconny's but I'll just keep trying different ways to handle escaping the path for the getimagesize() so it don't freak out... :D gosh an I was hoping to more on to the crop function... I have such a great idea on how to handle it... gaaa... lol

thanks
Jeremy
JeremyBASS

Re: Announcement: New plug-in SuperSizer

Post by JeremyBASS »

ok.. here I tested this with htmlspecialchars() and it works on the test bed with the spaces and all in the folders and all.. I bet this works for everyone.... bet/hope same thing  lol... -Jeremy
Attachments

[The extension txt has been deactivated and can no longer be displayed.]

darconny
Forum Members
Forum Members
Posts: 104
Joined: Thu Nov 22, 2007 12:43 pm

Re: Announcement: New plug-in SuperSizer

Post by darconny »

hmm..nope, the same.

I'm sorry for all this mess, don't know what's the problem on my site.
I used SuperSizer along with CGBlog, here is a part of the summary template:

{assign var=Photo value=$field->value}
            {assign var=Location value=$entry->file_location}
            {assign var=Item value=$entry->id}
            detail_url alt="{$entry->title}">
                {supersizer path="$Location/$Photo" height='80' Quality=100 Subdir="$Item"}
           

The last version before you implemented newest features works excellent. And now with the latest file I get this on the page:

warning:
getimagesize(http://localhost/cms-1.6.6/uploads/cgblog/id35/img 105.jpg) [function.getimagesize]: failed to open stream: HTTP request failed! HTTP/1.1 404 Not Found in C:\xampp\htdocs\cms-1.6.6\plugins\function.supersizer.php on line 46

Warning: Division by zero in C:\xampp\htdocs\cms-1.6.6\plugins\function.supersizer.php on line 257


What do you mean by showing you origin image paths, where should I look?  :)
JeremyBASS

Re: Announcement: New plug-in SuperSizer

Post by JeremyBASS »

well see it's looking for http://localhost/cms-1.6.6/uploads/cgblog/id35/img 105.jpg  so let's do this... can you get directly to http://localhost/cms-1.6.6/uploads/cgblog/id35/img 105.jpg  from the address bar... that tells me everything I need to know...

This is odd the test site http://www.digitalbarn.tv/testpage.html the image at the very bottom not only has a space in it but it also has a ' in it... so it should work for you...

I'm doing a job now so I'll have to come back to it later today... Thanks for the help

Jeremy
darconny
Forum Members
Forum Members
Posts: 104
Joined: Thu Nov 22, 2007 12:43 pm

Re: Announcement: New plug-in SuperSizer

Post by darconny »

yes I see the path.. I can get directly to that link and that opens source image in a page.

no problem.. thank you for all your work :)

Darko
Locked

Return to “Modules/Add-Ons”