?
avatar SoftForge
SoftForge
21 Jan 2017

Steps to reproduce the issue

Go to a field group overview, eg:

/administrator/index.php?option=com_fields&view=groups&context=com_users.user
/administrator/index.php?option=com_fields&view=groups&context=com_content.article

Click New

Expected result

The context is still in the URL.

Actual result

It is not in the URL 😭

System information (as much as possible)

Tested on Joomla 4.0-dev and staging.

Additional comments

This causes an ACL error if trying to login with a com_fields URL without a context.

avatar SoftForge SoftForge - open - 21 Jan 2017
avatar joomla-cms-bot joomla-cms-bot - change - 21 Jan 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 21 Jan 2017
avatar SoftForge SoftForge - edited - 21 Jan 2017
avatar SoftForge SoftForge - edited - 21 Jan 2017
avatar Bakual
Bakual - comment - 21 Jan 2017

It doesn't have to be in the URL. It is set in a userstate and taken from there afterwards.

This causes an ACL error if trying to login with a com_fields URL without a context.

If you want to use a direct link, you need to copy the initial link (eg the one from the sidebar). That is correct.

avatar Bakual Bakual - change - 21 Jan 2017
The description was changed
Title
Context is lost when adding a new field group
[com_fields] Context is lost when adding a new field group
Status New Closed
Closed_Date 0000-00-00 00:00:00 2017-01-21 23:19:47
Closed_By Bakual
avatar Bakual Bakual - close - 21 Jan 2017
avatar Bakual
Bakual - comment - 21 Jan 2017

Closing as expected behavior

avatar laoneo
laoneo - comment - 22 Jan 2017

Please reopen this one as it absolutely makes sense to have the context in the url similar to the fields or the extension parameter in com_categories.

avatar Bakual Bakual - change - 22 Jan 2017
Status Closed New
Closed_Date 2017-01-21 23:19:47
Closed_By Bakual
avatar Bakual Bakual - reopen - 22 Jan 2017
avatar Bakual
Bakual - comment - 22 Jan 2017

similar to the fields

For fields, the context (and for categories, the extension) isn't in the URL anymore as soon as you reload the list eg by applying sorting or filtering.
It's there for the create and edit tasks though. However those links don't work when used directly. You will get a "You're not allowed to access this page directly" error.

You can put it into the URL but I don't see the point having it there.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 5 Feb 2017

@SoftForge needs this Issue to stay open?

avatar Bakual
Bakual - comment - 5 Feb 2017

Imho not, but I'm not going to close it (again) 😄

avatar SoftForge
SoftForge - comment - 7 Feb 2017

I would say this is worth fixing. The reason is that you can click on a link (rather then posting a sort of filter) to get to this page. If you session has timed out, you actually get this login page which is good. However, when you login you get a nasty ACL error as the context is no longer in the userstate. I don't think you can create this situation by sorting or filtering (correct me if I'm wrong).

avatar Bakual
Bakual - comment - 7 Feb 2017

Easiest would be to implement a fallback to com_content.article as context if none is given. Then it would never fail.
Adding the URL in all cases would be more work.

avatar SoftForge
SoftForge - comment - 7 Feb 2017

That would be an improvement, although potentially confusing if you timed out adding fields to com_users and logged back in to arrive at a different set of fields.

avatar Bakual
Bakual - comment - 7 Feb 2017

Yep sure. But it would be kind of a similar behavior with how filtering works (where filters are reset when your session times out).

avatar franz-wohlkoenig franz-wohlkoenig - change - 30 Mar 2017
Category com_fields
avatar franz-wohlkoenig franz-wohlkoenig - change - 5 Apr 2017
Priority Medium Low
Status New Confirmed
avatar brianteeman
brianteeman - comment - 13 Jul 2017

@laoneo any thoughts on how to resolve this?

avatar laoneo
laoneo - comment - 17 Jul 2017

If I'm allowed to, then I would like to forward this issue to @Bakual, as he did the whole groups move from categories to their own entities.

avatar Bakual
Bakual - comment - 18 Jul 2017

I'm going to quote my statement from earlier:

You can put it into the URL but I don't see the point having it there.

I already closed this issue as expected behavior. It's the same behavior com_categories has except that there is no fallback to com_content like com_categories seems to have.
Do you want that fallback? Even if it may be confusing to suddenly have com_content fields?

avatar franz-wohlkoenig franz-wohlkoenig - change - 19 Jul 2017
Status Confirmed Information Required
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 18 Aug 2017

If this PR get no Response, it will be closed at 17th September 2017.

avatar franz-wohlkoenig franz-wohlkoenig - change - 17 Sep 2017
Status Information Required Closed
Closed_Date 0000-00-00 00:00:00 2017-09-17 06:10:14
Closed_By franz-wohlkoenig
avatar joomla-cms-bot joomla-cms-bot - change - 17 Sep 2017
Closed_Date 2017-09-17 06:10:14 2017-09-17 06:10:15
Closed_By franz-wohlkoenig joomla-cms-bot
avatar joomla-cms-bot joomla-cms-bot - close - 17 Sep 2017
avatar joomla-cms-bot
joomla-cms-bot - comment - 17 Sep 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 17 Sep 2017

This has been closed due to lack of response to the requests above – it can always be reopened.

Add a Comment

Login with GitHub to post a comment