?
Referenced as Pull Request for: # 9037
avatar brianteeman
brianteeman
13 Dec 2015

Testing in current staging when creating a user note the field for the User is marked as required but you can still save a note with selecting a user - making the whole thing useless

Previously it was not possible to save a user note without completing the required field

avatar brianteeman brianteeman - open - 13 Dec 2015
avatar brianteeman
brianteeman - comment - 28 Jan 2016

It seems to be ignoring the required="true" in the note.xml


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

avatar infograf768
infograf768 - comment - 28 Jan 2016

Maybe this needs a special case in the model in LoadFormData

avatar infograf768
infograf768 - comment - 29 Jan 2016

I think it comes from #5655

@dgt41 Please look at this.


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

avatar dgt41
dgt41 - comment - 29 Jan 2016

@infograf768 more likely it's coming from: #5655
I will look at this later on as I am awfully busy now

avatar brianteeman
brianteeman - comment - 29 Jan 2016

No that doesnt do anything - thanks for taking a look

On 29 January 2016 at 13:25, Dimitri Grammatikogianni <
notifications@github.com> wrote:

For testing try adding required" after

https://github.com/joomla/joomla-cms/blob/staging/administrator/templates/isis/html/layouts/joomla/form/field/user.php#L53


Reply to this email directly or view it on GitHub
#8667 (comment).

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar brianteeman brianteeman - change - 31 Jan 2016
Status New Closed
Closed_Date 0000-00-00 00:00:00 2016-01-31 17:44:42
Closed_By brianteeman
avatar brianteeman
brianteeman - comment - 31 Jan 2016

Closed as we have a Pr #9037


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

avatar brianteeman brianteeman - close - 31 Jan 2016

Add a Comment

Login with GitHub to post a comment