?
Referenced as Pull Request for: # 10522
avatar BurtNL
BurtNL
16 May 2016

It looks like it is possible to save an article or category without a valid name, while working in the modal view.

Steps to reproduce the issue

Install latest staging and multilanguage functionality, since the tab Associations is needed for this test.
Make sure you have 2 articles or 2 categories in 2 different languages.
Open an article and go to tab Associations. Make sure the Edit-button is present.

screen shot 2016-05-16 at 15 48 21

Edit the associated article via that Edit-button, clear the name of the article and click 'Save'.
Saving is not possible after this click, a red warning will be shown, as expected.

Click 'Save and Close'. Now the article will be saved with an empty name and no warning will be shown.

Expected result

A warning should have been showed: Invalid field: name.

Actual result

No warning about the missing name of the article, the article can be saved.
As a result the article is locked in the overview of articles.

System information (as much as possible)

Windows 10 / WAMP

Additional comments

This issue seems to be present in all modal views while editing a topic in such a view.

avatar BurtNL BurtNL - open - 16 May 2016
avatar andrepereiradasilva
andrepereiradasilva - comment - 16 May 2016

was able to reproduce in categories associations @JoomliC

avatar JoomliC
JoomliC - comment - 16 May 2016

Ok!
Confirmed with module edit, and issue already there in... 3.5.1!

In fact, you can test with 3.5.1 (before i did any change based on module edit modal) with module edit, and the issue was already there. (Go to Menus > open a menu > go to Modules Assignment tab, and then click on one module name to edit the module, and do as here, click on Save & Close after deleting the module title : same issue).
Not related to my change, but of course we need to fix it! ;-)

I will try to find a good solution to prevent such possible issue. (a way to not close the modal, if an error occures).
Or... to remove the save & close, to keep only close and save buttons ?

avatar brianteeman brianteeman - change - 16 May 2016
Category Administration Administration Multilanguage
avatar brianteeman brianteeman - change - 16 May 2016
Status New Confirmed
avatar JoomliC
JoomliC - comment - 16 May 2016

@BurtNL Tested, no "visible" error message, and article locked, but i can't save it with an empty name (the previous name is still there).
So did you really get an empty name after saving ? (article ? category ? module ?)

avatar JoomliC
JoomliC - comment - 16 May 2016

was able to reproduce in categories associations

@andrepereiradasilva with your test with category edit in association, with Save & Close, and with latest staging (as categories modals merged) are you able to save with a empty title (i mean the title is really empty in database!) or it close, but in reality, nothing was changed in database ?
(which would mean a better behavior than in 3.5.1, where it's possible to really save an empty title for module in database, but with alreay changes done in 3.6.0-dev, this is not anymore possible for me (with the previous fixes)).

Thanks for help! ;-)

avatar andrepereiradasilva
andrepereiradasilva - comment - 16 May 2016

the title is not changed.

avatar JoomliC
JoomliC - comment - 16 May 2016

@andrepereiradasilva Thanks! (that means a better behavior than 3.5.1 when in modal)
I've got maybe an idea on how to not close if any error... Maybe will check this tomorrow ;-)

avatar JoomliC JoomliC - reference | 121e5ba - 17 May 16
avatar JoomliC JoomliC - reference | 2165d63 - 17 May 16
avatar JoomliC JoomliC - reference | 4897400 - 17 May 16
avatar JoomliC JoomliC - reference | 2e2af7c - 17 May 16
avatar JoomliC
JoomliC - comment - 17 May 2016

@BurtNL @andrepereiradasilva Just created the PR with fix for category, newsfeed and contact edit modal : #10522
Note: as i have other changes to update article edit, and to do a code review for module, i will add this patch in the same time, in separated PRs (to come this week of course, to have all done for 3.6.0 ;-) )
Thanks for reporting this issue, and testing! ????

Note: I think we can close here as we have a PR ;-)

avatar brianteeman brianteeman - close - 17 May 2016
avatar brianteeman brianteeman - change - 17 May 2016
Status Confirmed Closed
Closed_Date 0000-00-00 00:00:00 2016-05-17 08:30:11
Closed_By brianteeman
avatar brianteeman
brianteeman - comment - 17 May 2016

Closed as we have a PR #10522


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

avatar brianteeman brianteeman - close - 17 May 2016

Add a Comment

Login with GitHub to post a comment