[SOLVED] Image alt, caption, cost & some more extra fields if possible

vineonardo · 2012-08-07 15:16:39

Hello,
I need to add some additional fields to pre-existing fields that come with txp’s image form.
After the alt & caption, I want to add a simple field called as cost and another one as product-id.
These two will go in “txp_image” table’s cost & product_id columns respectively.

I tried tweaking txp_image.php & txplib_db.php files a bit, it added “cost” field in the images form (check here). But when I try to save it, it gives error(check here).

I know getting it done this way is “wrong” but I just need to have some additional fields in the images form, as for this website, the images are working as individual products.

Thanks for your time.

EDIT:
This question is SOLVED.

~~I still have no clue to how to set this as ‘SOLVED’, it should be simple, they should add a button like ‘Set as SOLVED’ somewhere! :p~~

Nevermind, Thanks to everyone and especially Uli for helping me solve it. :)

Last edited by vineonardo (2012-08-07 20:51:03)

colak · 2012-08-07 15:59:03

Hi Vineet

couldn’t you use the “image name” and the alt or caption fields to do what you need?

vineonardo · 2012-08-07 16:19:17

Hi Colok,
Thanks for your reply. The thing is, all the fields except for the caption are being used to display product information. I’m only left with caption field right now which I can use but in future I might need more fields; also, caption can be alphanumeric but the cost has to be numeric only. This kind of validation is important as in future someone else might be updating the website.

We are trying to connect this installation of the textpattern to an e-commerce application, for products we are using images table. Thus we need to have more fields in image form only so that adding and updating the products will be easier and single time job.

Last edited by vineonardo (2012-08-07 16:21:28)

uli · 2012-08-07 18:19:02

Things like these are fun to me :)

Alright, let’s see whether we manage to succeed here in the forum, and not via file exchange, so that others might recap this.

I added a table row cost to txp_image table via phpMyAdmin: varchar(12), utf8_unicode_ci.

The following changes all go into txp_image.php (at the time of writing v4.4.1. And you best use a fresh, unaltered, copy):

What you’ll have done is at least a change around (around! original line numbers are now shifted!) line 490, so that the field is displayed on the Image panel. (I duplicated the alt line and changed all instances of alt -> cost)

Duplicate around line 122 (the one containing alt) and change alt -> cost (2x)
If you want to search for a price in the image list view, duplicate around line 306, again changing alt -> cost (2x)
Around line 638 look for extract(doSlash(gpsa(array('id','category','caption','alt' Add ,'cost' to the end (1x)
Around line 654 duplicate the line containing alt and change alt -> cost (2x)

That should be sufficient, I think. Seems to work, at least here, and as far as I tried.

I just don’t see why you thought you’d have to edit txplib_db.php, these functions there are too basic, too little specialised, IMHO.

Last edited by uli (2012-08-07 18:45:00)

vineonardo · 2012-08-07 18:34:56

@uli
Thanks for your answer. Duplicating alt line was the first thing I did, but due to lack of my php knowledge, I couldn’t get too far with that. I’ll follow your instructions with a fresh copy of textpattern again. :D

Thanks again, TextPattern’s forum members are so helpful, I knew someone would surely guide me. I am really very grateful.

bici · 2012-08-07 19:05:06

uli wrote:

What you’ll have done is a change around line 490, so that the field is displayed on the Image panel. (I duplicated the alt line and changed all instances of alt -> cost)

And will this change have to be made every time one updates Textpattern?

uli · 2012-08-07 19:27:27

Yes.

vineonardo · 2012-08-07 19:35:42

@uli

I did as you told, but additionally I also updated some more lines like

duplicating alt line at line number 490
at line number 231, I added 'cost='.urlencode($cost).a..

It works and now changes are also reflecting in the database just as they had to, thanks again.

What I was doing wrong at first time was, setting “Cost” as INT in database instead of VARCHAR as you suggested.

Thanks again, really appreciate your time & efforts! :)

p.s. Just in-case anyone wants have a copy of the file that I tweaked, you can download it here

uli · 2012-08-07 19:47:29

You’re welcome, it was fun fiddling around a little :)

vineonardo wrote:

I did as you told, but additionally I also updated some more lines like

duplicating alt line at line number 490

at line number 231, I added 'cost='.urlencode($cost).a..

Interesting: I had these included as a sweeping blow at the beginning but found I could leave them out as well. What are they used for, Vineet, did you find out?

Last edited by uli (2012-08-07 19:49:56)

vineonardo · 2012-08-07 19:55:03

uli wrote:

What are they used for, Vineet, did you find out?

Haha, well, like I said before, I’m not a PHP expert, so I can’t really give a good logical answer for your question, although, I’d like an answer for that too.

Do you think that line number 231’s code, 'cost='.urlencode($cost).a. helps textpattern to put the given data in the database table?

I guess that’s what it does. I think so cause, I was playing with this for over 2 hours before I posted question and I was getting errors, which now I guess, was cause of using INT instead of VARCHAR.

EDIT:

p.s How do I mark this post as ‘SOLVED’ ? :p

Last edited by vineonardo (2012-08-07 19:55:36)

uli · 2012-08-07 20:03:05

vineonardo wrote:

Do you think that line number 231’s code, 'cost='.urlencode($cost).a. helps textpattern to put the given data in the database table?

To be honest: I don’t think it does, as it worked here without this change. I’m curious as well for the PHP experts to return from their barbecues/vacations ;)

vineonardo wrote:

p.s How do I mark this post as ‘SOLVED’ ? :p

Just edit the first post. Thanks!

vineonardo · 2012-08-07 20:39:42

I was wondering if I could retrieve the ‘cost’ using <txp:image_info type="cost" />,
so I update the taghandlers.php file.

It works!

Textpattern CMS

Textpattern CMS support forum

#1 2012-08-07 15:16:39

[SOLVED] Image alt, caption, cost & some more extra fields if possible

#2 2012-08-07 15:59:03

Re: [SOLVED] Image alt, caption, cost & some more extra fields if possible

#3 2012-08-07 16:19:17

Re: [SOLVED] Image alt, caption, cost & some more extra fields if possible

#4 2012-08-07 18:19:02

Re: [SOLVED] Image alt, caption, cost & some more extra fields if possible

#5 2012-08-07 18:34:56

Re: [SOLVED] Image alt, caption, cost & some more extra fields if possible

#6 2012-08-07 19:05:06

Re: [SOLVED] Image alt, caption, cost & some more extra fields if possible

#7 2012-08-07 19:27:27

Re: [SOLVED] Image alt, caption, cost & some more extra fields if possible

#8 2012-08-07 19:35:42

Re: [SOLVED] Image alt, caption, cost & some more extra fields if possible

#9 2012-08-07 19:47:29

Re: [SOLVED] Image alt, caption, cost & some more extra fields if possible

#10 2012-08-07 19:55:03

Re: [SOLVED] Image alt, caption, cost & some more extra fields if possible

#11 2012-08-07 20:03:05

Re: [SOLVED] Image alt, caption, cost & some more extra fields if possible

#12 2012-08-07 20:39:42

Re: [SOLVED] Image alt, caption, cost & some more extra fields if possible

Board footer