? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
22 Dec 2017

Steps to reproduce the issue

Enable profile plugin. set fields to required for site and admin edit. We used Phone, Company, and website in our test.

Expected result

Profile plugin fields are required during admin edit.

Actual result

Can save profile during admin edit when profile plugin fields are empty.

After this PR the fields set to required are marked with a * and have the class required

PR for #14805

avatar brianteeman brianteeman - open - 22 Dec 2017
avatar brianteeman brianteeman - change - 22 Dec 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 22 Dec 2017
Category Front End Plugins
31f79d2 22 Dec 2017 avatar brianteeman tab
avatar brianteeman brianteeman - change - 22 Dec 2017
Labels Added: ?
3ec6bd0 22 Dec 2017 avatar brianteeman oops
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 23 Dec 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 23 Dec 2017

I have tested this item successfully on 634f1d6


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

avatar Quy Quy - test_item - 27 Dec 2017 - Tested successfully
avatar Quy
Quy - comment - 27 Dec 2017

I have tested this item successfully on 634f1d6


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 27 Dec 2017
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 27 Dec 2017

Ready to Commit after two successful tests.

avatar mbabker mbabker - change - 3 Jan 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-01-03 00:12:03
Closed_By mbabker
Labels Added: ?
avatar mbabker mbabker - close - 3 Jan 2018
avatar mbabker mbabker - merge - 3 Jan 2018
avatar brianteeman
brianteeman - comment - 3 Jan 2018

Thanks

Add a Comment

Login with GitHub to post a comment