custom attributes not saving

[mferguson]

In version 3.03 I have located a bug that prevents custom fields from being saved on product upload. The scenario is this:

1) Select Upload Ad - for a category that has several custom fields (I have tested only text fields)

2) Fill out the fields and submit

3) Resulting ad does not have the values entered displayed in the fields


Please note that this can be an intermittent problem...here is why:

In uploadproduct.php there are the following lines:

Content visible to verified customers only.

Nowhere in this file is $i initialized to a value. Given this it may equate to empty or a random value. In my case rather than showing up as empty (therefore enabling the following strstr function to operate correctly) $i is set to a random value preventing a match using the strstr function. This prevents the attributes from being saved.

In my case I removed $i from the above lines and the attributes/custom fields are now saved correctly.

Thanks!

Mark

[Chuck S]

Of course it is initialized as the fields are already defined in the page as an array. Your simply doing a foreach custom- array return results and the code is quite clear here.

PhotoPost Classifieds - test

I am not seeing any issues and I tested every type of extra field there is

I would suggest you email support with enough urls and logins ftp etc database and explain your issue. If you change the code your not going to be able to save your array results.

[mferguson]

In the use of the variable $i relative to $thesevars it has no relation to an array coming in as part of the post variables. I know how arrays are used and indexed but in this case I don't see where $i is initialized to some value or being incremented in some way prior to where the strstr is being done. If it I'm missing where this is done please let me know. Thanks

Mark

[Chuck S]

Look at your template as well and the uploadproduct file they are all initialized as arrays and the code works as posted and shown by my example.

If your happy you think your change works then okay. I was just merely pointing out there should be no need to change code. $i is nothing relatively speaking just a beautification or reminder we are processing an array. $thesevars is set to custom- which is indeed correct.

Content visible to verified customers only.

All the variables for the customs are an array defined by custom- and we do a foreach.