Announcement: New plug-in SuperSizer

Have a question or a suggestion about a 3rd party addon module or plugin?
Let us know here.
Locked
JeremyBASS

Re: Announcement: New plug-in SuperSizer

Post by JeremyBASS »

Why thanks.. I surely was not complaining ... but thank you for sure :D cheers -Jeremy
rubberglover
Forum Members
Forum Members
Posts: 53
Joined: Mon Jul 23, 2007 5:30 pm

Re: Announcement: New plug-in SuperSizer

Post by rubberglover »

haha, I know you weren't complaining, but you've saved my clients (and therefore me!) an awful lot of trouble so it's the least I can do :)
kendo451

Re: Announcement: New plug-in SuperSizer

Post by kendo451 »

Trying to use this with the News module, news gives $entry->file_location as a URL instead of a path:

http://domain.com/uploads/news/entryid

Supersizer wants it in the form of a relative path from the root.

Other than writing a UDT to strip the domain off the $entry->file_location does anyone have a better idea?
calguy1000
Support Guru
Support Guru
Posts: 8169
Joined: Tue Oct 19, 2004 6:44 pm
Location: Fernie British Columbia, Canada

Re: Announcement: New plug-in SuperSizer

Post by calguy1000 »

Uploads does the same thing (provides the url to the image).
Follow me on twitter
Please post system information from "Extensions >> System Information" (there is a bbcode option) on all posts asking for assistance.
--------------------
If you can't bother explaining your problem well, you shouldn't expect much in the way of assistance.
JeremyBASS

Re: Announcement: New plug-in SuperSizer

Post by JeremyBASS »

Um no I took care of that.. you can use a full url.. here is the test..

http://www.corbensproducts.com/users/1/ ... stStripURL

that code for that is example as it sits there is...

Code: Select all

<a name="testStripURL"></a>
 <h3>Strip URL path test</h3>

{capture assign="testStripURL"}http://www.corbensproducts.com/uploads/images/IMG_5127(1).jpg{/capture}
<pre>{$testStripURL|escape:'htmlall'}</pre><br/>
{supersizer path=$testStrip height="400" alt="SMOOTH" subdir=$oneuser.id quality=55 filter="SMOOTH" arg_1=50 suffix="testStripURL" fill_attr=true strip_tags=true}

it does try to auto strips that but for a better way just use the  strip_tags=true param.. that is a much more bullet proof way.. hope that helps.. Cheers
kendo451

Re: Announcement: New plug-in SuperSizer

Post by kendo451 »

Yes, strip_tags=true did the trick!
kendo451

Found a bug?

Post by kendo451 »

I found a bug, I think.  At least it is an inconsistent performance.

This is using the latest version that you just posted.

When you feed the path= a URL instead of a relative path, then it fails, even when strip_tage=true.

See attached screenshot for the output.
Attachments
ss_screenshot1.jpg
Last edited by kendo451 on Wed Jul 14, 2010 6:56 am, edited 1 time in total.
kendo451

Definitely a bug

Post by kendo451 »

Ok, I have definitely confirmed it is a bug, but only when you feed the path a full url.  strip_tags=true isn't solving the problem.
Last edited by kendo451 on Wed Jul 14, 2010 6:46 am, edited 1 time in total.
JeremyBASS

Re: Announcement: New plug-in SuperSizer

Post by JeremyBASS »

hmm.. I'll take a look into it. tk -J
JeremyBASS

Re: Announcement: New plug-in SuperSizer

Post by JeremyBASS »

Ok.. please try this one.. it should match better now as it has this regex

[http|https]://(.*?)/(.*?)

tested here
http://www.corbensproducts.com/users/1/ ... stStripURL


also I have the scale in for the crop.. tesded here

http://www.corbensproducts.com/users/1/ ... #cropscale


Let me know if there is still an issue please.. Cheers
JeremyBASS

Re: Announcement: New plug-in SuperSizer

Post by JeremyBASS »

Helps if I put the file up I requestion the test on :D .. tk
Attachments

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

kendo451

Re: Announcement: New plug-in SuperSizer

Post by kendo451 »

Actually, Jeremy, if you look at the filepath it printed in the screenshot I attached above, you can see the problem.  It is splitting the full url on the // instead of splitting it on the slash that follows the domain.

I'll test this new version and get back to you today.
JeremyBASS

Re: Announcement: New plug-in SuperSizer

Post by JeremyBASS »

Yeah I figured that it was only getting the http:// there from your screen shot.. It turned out that what I posted didn't have the extra test for the full url.. I must have dropped it. but this last post does and should restore that feature.  Let me know if there is an issue.. the test atm shows it to be working so it should be ok.. cheers -Jeremy
JeremyBASS

Re: Announcement: New plug-in SuperSizer

Post by JeremyBASS »

I have updated the params to go with this next release..

the wiki docs

I just need a confirmation from kendo451 or anyone else that the strip_tag solved the http://domain.tld/ issue that some modules output with the image path.
Last edited by JeremyBASS on Wed Jul 14, 2010 5:58 pm, edited 1 time in total.
User avatar
NikNak
Forum Members
Forum Members
Posts: 183
Joined: Fri Oct 02, 2009 2:28 pm

Re: Announcement: New plug-in SuperSizer

Post by NikNak »

Hi Jeremy
Post shortened (not helpfull to anyone) - I was being dumb!

using:{supersizer path=$pic|replace:$gCms->config.root_url:'' width='190'  height='95' URL=true protect=0 crop='true' subdir='people' assign="thumbpic" passthru="true" quality=90 crop="center,center" debug=true }

outputs no image. I also tried the striptags  with path=$pic but that really messed the path up.

Nik
Last edited by NikNak on Thu Jul 15, 2010 3:44 pm, edited 1 time in total.
Locked

Return to “Modules/Add-Ons”