?
avatar PhilETaylor
PhilETaylor
27 Mar 2017

Steps to reproduce the issue

Install joomla-cms/staging@1c52ed3

Create imagelist custom field, only completing the required fields such as:

screen shot 2017-03-27 at 15 06 26

Go to front end and edit article to see the dropdown, select an image and click save
or
Go to back end and edit article to see the dropdown, select an image and click save

Expected result

The selected value of the dropdown is saved.

Actual result

The selected value of the dropdown is NOT saved.

see: http://screenshot.myjoomla.io/2M2d1S0C3t15

System information (as much as possible)

https://gist.github.com/PhilETaylor/b1ff259a06518f903339950ca81e35d7

avatar PhilETaylor PhilETaylor - open - 27 Mar 2017
avatar joomla-cms-bot joomla-cms-bot - change - 27 Mar 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 27 Mar 2017
avatar brianteeman
brianteeman - comment - 27 Mar 2017

Couldnt get your recording to load :(

Can not confirm on the admin see
phil2

avatar brianteeman
brianteeman - comment - 27 Mar 2017

And cannot confirm on frontend either

phil3

avatar PhilETaylor
PhilETaylor - comment - 27 Mar 2017

Thats because you are superman and every one else is just an idiot.

avatar PhilETaylor
PhilETaylor - comment - 27 Mar 2017

Infact at the current git revision NONE OF THE CUSTOM FIELDS SAVE!!!

avatar PhilETaylor PhilETaylor - edited - 27 Mar 2017
avatar PhilETaylor
PhilETaylor - comment - 27 Mar 2017

Full video: https://www.youtube.com/watch?v=FveVUozZMPc

This video shows me installing from github from d7caa77 revision and results in NO fields being saved

avatar PhilETaylor PhilETaylor - change - 27 Mar 2017
Title
com_fields imagelist value not saving
com_fields value saving to db but then not being retrievable
avatar PhilETaylor PhilETaylor - edited - 27 Mar 2017
avatar AlexRed
AlexRed - comment - 27 Mar 2017

I can't confirm the issue


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14932.

avatar PhilETaylor
PhilETaylor - comment - 27 Mar 2017

Changed title to be

com_fields value saving to db but then not being retrievable

Im still testing but I think this commit broke it 1c52ed3 which is from PR #14558

avatar PhilETaylor
PhilETaylor - comment - 27 Mar 2017

@AlexRed Can you confirm your PHP version and the EXACT COMMIT that you are testing against?

avatar brianteeman
brianteeman - comment - 27 Mar 2017

Probably #14558

avatar brianteeman
brianteeman - comment - 27 Mar 2017

i can confirm this on current staging (nightly wont show it as #14558 was merged today)

avatar brianteeman brianteeman - change - 27 Mar 2017
Labels Added: ?
avatar brianteeman brianteeman - labeled - 27 Mar 2017
avatar PhilETaylor
PhilETaylor - comment - 27 Mar 2017

Probably #14558

Like I said...

This is why testing should be done on the very latest code available - not nightly....

So Im not an idiot after all.....

avatar brianteeman
brianteeman - comment - 27 Mar 2017

No you are still an idiot

avatar infograf768
infograf768 - comment - 27 Mar 2017

I confirm that a Text custom field duely filled in admin does not display in frontend

avatar AlexRed
AlexRed - comment - 27 Mar 2017

Also my test in nightly.

avatar PhilETaylor
PhilETaylor - comment - 27 Mar 2017

@infograf768 I confirm that a Text custom field duely filled in admin does not display in frontend

or backend you will find

@AlexRed Also my test in nightly.

Thanks - thats what I guessed.

@brianteeman No you are still an idiot

Personal attacks (should) not be tolerated by the Joomla project. Even in jest (if such). Being an active contributor doesn't give you a right to issue personal attacks. Even those in power can be removed... repeatedly.

avatar allrude
allrude - comment - 27 Mar 2017

What You Give Is What You Get

avatar joomdonation
joomdonation - comment - 27 Mar 2017

In my test, the custom fields data is not being saved. Checked the code and found typo. I made the PR #14941 which should solve this issue.

avatar PhilETaylor
PhilETaylor - comment - 27 Mar 2017

Fixed by #14941

avatar PhilETaylor PhilETaylor - change - 27 Mar 2017
Status New Closed
Closed_Date 0000-00-00 00:00:00 2017-03-27 16:24:53
Closed_By PhilETaylor
avatar PhilETaylor PhilETaylor - close - 27 Mar 2017
avatar zero-24 zero-24 - change - 16 Oct 2017
Labels Removed: ?
avatar zero-24 zero-24 - unlabeled - 16 Oct 2017

Add a Comment

Login with GitHub to post a comment