?
avatar waader
waader
18 Dec 2016

Steps to reproduce the issue

With current staging I added two field groups with a text field assigned to each group. I also added a category that I assigned to both fields.

Then I added an article and assigned it the newly created category. That went fine but when I added another article with the newly created category I got this error:

0 Invalid controller: name='article', format=''

Tested with mysql.

Expected result

Actual result

System information (as much as possible)

Additional comments

avatar waader waader - open - 18 Dec 2016
avatar joomla-cms-bot joomla-cms-bot - change - 18 Dec 2016
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 18 Dec 2016
avatar Bakual
Bakual - comment - 18 Dec 2016

Can you tell us what URL you had in the browser when that error occured? Sounds unrelated to fields to me because com_fields doesn't deal with URLs but this error comes from a wrong URL.

avatar waader
waader - comment - 19 Dec 2016

Today I tried to reproduce the error which proved to be not an easy task. Finally I succeeded. When adding an article I selected a category and I had to hit immediatly the "Save & Close" Button afterwards. Now the error is thrown. And the URL is ...

http://xyz/joomla37/administrator/index.php?return=aHR0cDovL2xvY2FsaG9zdC9qb29tbGEzNy9hZG1pbmlzdHJhdG9yL2luZGV4LnBocD9vcHRpb249Y29tX2NvbnRlbnQmdmlldz1hcnRpY2xlJmxheW91dD1lZGl0&option=com_fields&task=field.storeform&context=com_content.article&formcontrol=jform

avatar Bakual
Bakual - comment - 19 Dec 2016

Ah, if you select a category, I think com_field does a reload because the available fields may have changed. You interefered with that reload and got stuck between chair and table.
@laoneo Do you think that is correct?
We could maybe add the "loading spinner" so it is apparent that something is happening?

avatar laoneo
laoneo - comment - 20 Dec 2016

Yes it is correct, when you hit the save button before the page got reloaded, then you can land it that error.

avatar tonypartridge
tonypartridge - comment - 21 Dec 2016

I Suggest we disable save button on category change and enable on category change finished?

avatar ggppdk
ggppdk - comment - 21 Dec 2016

I Suggest we disable save button on category change and enable on category change finished?

About disabling save button, or hiding the toolbar, agreed it is an option
but there is no "category change finished",

because the form submits
and a new page is loaded by the browser, old page is gone

It appears as if the page remains the same, but actually it is a new page, a new form that loads the values that had been entered by the user


Solution, just hide the outer form container and append a div under body with a loading message
then continue with the form submission

Or just add a grayed overlay over the form with a centered loading message

e.g. the following can be shown till the new page appears

loading_please_wait

avatar tonypartridge
tonypartridge - comment - 21 Dec 2016

Clearly in had not tested it fully, in which case we should prompt a
loading screen in a modal with a grey out background it will be cleared on
the new page load

On Wed, 21 Dec 2016 at 19:20, Georgios Papadakis notifications@github.com
wrote:

I Suggest we disable save button on category change and enable on category
change finished?

About disabling save, or hidding the toolbar agreed it is an option

but there is no "category change finished",

because the form submits

and a new page is loaded by the browser, old page is gone

It appears as if the page remains the same, but actually it is a new page
a new form that loads the values that had been entered by the user


Solution, just hide the outer form container and append a div under body
with a loading message

then continue with the form submission

Or just add an grayed overlay over the form with a centered loading message

[image: loading_please_wait]
https://cloud.githubusercontent.com/assets/5092940/21401891/af4e85e0-c7be-11e6-8322-f49f7383d3db.png


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#13267 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABVgluTQhcZrmnEbURp56ME-i6UNhdDKks5rKXvygaJpZM4LQKnK
.

avatar laoneo
laoneo - comment - 21 Dec 2016

Can we add a UX label here that they can come up with a proper solution?

avatar ggppdk
ggppdk - comment - 21 Dec 2016

I have made a PR please click on it to see the picture and comment on it

e.g.
comment about
-- place to add styles, or tell me if an overlay CSS class already exists in Joomla
-- better message text

avatar zero-24 zero-24 - close - 21 Dec 2016
avatar zero-24
zero-24 - comment - 21 Dec 2016

Closing as we have Code to comment on. Thanks!

avatar zero-24 zero-24 - change - 21 Dec 2016
Status New Closed
Closed_Date 0000-00-00 00:00:00 2016-12-21 20:16:44
Closed_By zero-24
avatar zero-24 zero-24 - close - 21 Dec 2016

Add a Comment

Login with GitHub to post a comment