Images are 1st class citizens of TXP

jsoo · 2010-10-03 16:09:50

Stef: noticed that thumbnail and image do a database fetch even when using $thisimage. Am I missing something, or is this a lot of unnecessary database calls? That is, is there some case where $thisimage gets an image id but not the rest of the data? I looked but didn’t see any such.

Edit: That is, I’m happy to submit a patch for this; just wanted to check first that I haven’t missed something.

Last edited by jsoo (2010-10-03 16:17:19)

Bloke · 2010-10-04 12:54:29

jsoo wrote:

noticed that thumbnail and image do a database fetch even when using $thisimage

Yikes! Good catch. What a potential waste of resources *slaps self*.

I just swapped the $rs = safe_row(...); junk for $rs = $thisimage; and the only difference I can make out is that the date is now returned in its epoch time instead of its database-encoded form. Effects:

zero difference to the <txp:image_info /> tag because it returns the raw (UNIX epoch) date anyway
zero difference on the <txp:image_date /> tag because it returns a user-overridable formatted date
zero difference on previous <txp:image /> and <txp:thumbnail /> implementations because a) $thisimage didn’t exist, b) you couldn’t output the image date via a native tag
zero difference on plugins such as upm_image because it would have to manually fetch additional date info since $thisimage didn’t exist
shedload of difference in performance when dealing with lists of images

I’m glad you’re here. You keep my stupidity in check. Many thanks, will commit patch asap.

Last edited by Bloke (2010-10-04 12:55:37)

jsoo · 2010-10-13 20:00:04

image_list has ‘extensions’ in the array it sends to lAtts(), but then checks against $atts['ext'].

Bloke · 2010-10-13 20:07:01

jsoo wrote:

image_list has ‘extensions’ in the array it sends to lAtts(), but then checks against $atts['ext'].

^{/me slaps forehead}

Schoolboy error, thanks and fixed. Note to self: when searching for variables to replace, don’t just use $ext… *sigh*

mrdale · 2010-11-03 00:30:41

I wonder wtf I’m doing wrong? I can’t get image_list to work at all…

calling it with the stock example. I must be doing something really stupid.

Tag error: <txp:image_list category="facility"> ->  Textpattern Warning: tag does not exist  on line 1183

textpattern/publish.php:1183 trigger_error()
textpattern/publish.php:1104 processTags()
textpattern/publish.php:535 parse()
index.php:45 textpattern()

Last edited by mrdale (2010-11-03 00:30:57)

hidalgo · 2010-11-03 00:36:43

It looks like the <txp:image_list /> tag has been renamed to <txp:images /> in r3443

Gocom · 2010-11-03 02:56:45

hidalgo wrote:

It looks like the <txp:image_list /> tag has been renamed to <txp:images /> in r3443

Yeah. It’s kinda annoying that the function name is already used. Well, now the tag is shorter which is plus. Maybe slightly out of the place when the other tags have the _list, but it kinda does fit better with article tags now.

Better to be safe than sorry with function declarations.

Last edited by Gocom (2010-11-03 02:57:06)

mrdale · 2010-11-03 16:23:36

thanks. maybe the examples could use an edit?

damienbuckley · 2010-11-16 02:35:40

Unless I’m using it wrong, ‘offset’ doesn’t work. Limit works fine

<txp:images break=“li” offset=“1” limit=“1”><txp:image class=“frame” /></txp:images>@@

I’ve tried all manner of permutations with this and there’s no way I can seem to skip the first image in the article_image list.

hidalgo · 2010-11-16 03:10:14

If it’s outputting a single image but not the correct one, it’s probably because <txp:images /> doesn’t maintain the sort order of the article_image list.

damienbuckley · 2010-11-16 03:17:51

Yeah I noticed that. It seems to show the first image regardless. I’m using the 1st image for a banner and the rest to list logo’s down the side.

I’m back to using hak_article_image for now. Hopefully they can fix this shortly. Not that it makes a lot of difference – hak_article_image works fine – I’d just prefer the native behaviour to work properly.

hidalgo · 2010-11-16 03:37:03

Yeah, I hope it can be fixed in the future also. It would be much more useful for situations like you’ve mentioned.

Textpattern CMS

Textpattern CMS support forum

#121 2010-10-03 16:09:50

Re: Images are 1st class citizens of TXP

#122 2010-10-04 12:54:29

Re: Images are 1st class citizens of TXP

#123 2010-10-13 20:00:04

Re: Images are 1st class citizens of TXP

#124 2010-10-13 20:07:01

Re: Images are 1st class citizens of TXP

#125 2010-11-03 00:30:41

Re: Images are 1st class citizens of TXP

#126 2010-11-03 00:36:43

Re: Images are 1st class citizens of TXP

#127 2010-11-03 02:56:45

Re: Images are 1st class citizens of TXP

#128 2010-11-03 16:23:36

Re: Images are 1st class citizens of TXP

#129 2010-11-16 02:35:40

Re: Images are 1st class citizens of TXP

#130 2010-11-16 03:10:14

Re: Images are 1st class citizens of TXP

#131 2010-11-16 03:17:51

Re: Images are 1st class citizens of TXP

#132 2010-11-16 03:37:03

Re: Images are 1st class citizens of TXP

Board footer