PDA

View Full Version : Save custom fields choices when errors


HenrikHansen
10-12-2009, 02:40 PM
If a link is submitted with errors, fx because the title has too many fields,
an error message shows up. All user submitted info is still shown, including tags, descriptions etc, except the custom fields. All choices of custom fields have been erased.

I hope in a future version it will be possible to save custom fields info in case of error messages.

Brian
10-13-2009, 01:19 PM
Custom field values certainly should be retained... Which version of vBa Links are you running exactly?

Our Sponsors
 

HenrikHansen
10-13-2009, 06:28 PM
Hi Brian

I am using Links 3.2.0

Try go to this site

http://www.europanet.dk/links/ferie-i-europa-606/

and choose add link "tilføj link" (no registration needed).

Submit more than 55 characters in title field and select some custom fields.
An error will occur and custom fields selections are gone.

Brian
10-15-2009, 01:21 PM
Hrmm... I definitely see the issue on your site, but for some reason I'm not able to reproduce it locally. Would you mind submitting a support ticket so I can take a look at things and see what the problem is?

Our Sponsors
 

HenrikHansen
10-20-2009, 01:00 PM
OK, submitting support ticket.

BWJ
10-24-2009, 08:25 PM
I can confirm this error.

Lets say:

1) Title is required
2) custom field A is required
3) custom field B is required

While submitting a link - if you forget:

1), 2) and 3) = no error message
1) and 2) = error message
1) = double error message
2) and 3) = no error message
2) = error message
3) = error message
1) and 3) = error message

And the error message is the same all the way: Warning: unserialize() expects parameter 1 to be string, array given in [path]/includes/vba_links_functions_savelink.php on line 423

I would be nice to hear if any solution was found.

HenrikHansen
10-25-2009, 04:20 AM
Brian says in support ticket he has fixed a very small bug in the addlink.php file.
It is uploaded in the download package.

BWJ
10-26-2009, 07:58 AM
It seems to work now. Thank you Henrik! That was an important bug to get rid of.