? ? ? Pending

User tests: Successful: Unsuccessful:

avatar chmst
chmst
18 Dec 2020

Pull Request for Issue #31690 ., #32270

Summary of Changes

  • Fix double id=id=
  • Change url in banner module as required.
  • Use camelCaps for a variable catIds.
  • Change the title "Admin Menu" to "Special Menu".
  • Change the the "Author Login" to "Administrator Login" in Text.
  • Use access levels for changing login / logout links in top menu and footer menu
  • Don't enable workflow automatically
  • Small code improvements

Testing Instructions

Make a new installation of j4 and install the patchtester. (thanks @richard67)
To make testing easier: export the database (with drop table) , so you can import it for the second step.

Install Blog Sample Data in a fresh J4 and refresh the dashhboard view
Find all the issues described in "BEFORE Patch"

Apply the Patch.
Import now the database which you have saved (or make a new installation)
Find all the resolved issues described in "AFTER Patch"

Actual result BEFORE applying this Pull Request

Backend:

  • In the menu "content", the menu item workflow is visible (after refresh), the workflow is enabled
  • There is a menuItem anf a module "Author menu"
  • There are some menuItems where the link contains "id=id="

Frontend:

  • The link in the banner to typography is not made ser friendly
  • In the top menu and bottom menu there is a link "login". It remains login, also after you logged in.
  • If the user logs out, a 404 occurs

Expected result AFTER applying this Pull Request

Backend:

  • No menuItem workflow is visible (after refresh)
  • Menu Item and module "Author menu" are now "Special menu"
  • In all menuItems "id=id=" is correct "id="
  • Check the menuItems in Admin Menu. There must be a consitent tree structure, no orphaned items

Frontend:

  • The link in the banner to typography is not made userfriendly.
  • If a user is logged in, the menu Items logout appear and vice versa.
  • If the user logs out, no error occurs
avatar chmst chmst - open - 18 Dec 2020
avatar chmst chmst - change - 18 Dec 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Dec 2020
Category Front End Plugins
avatar chmst chmst - change - 18 Dec 2020
Title
Fix links, use camelCaps for variable catIds
[4.0] Blog Sampledata: Fix links, use camelCaps for variable catIds
avatar chmst chmst - edited - 18 Dec 2020
avatar brianteeman
brianteeman - comment - 18 Dec 2020

The menu items are correct now but the typography link is now index.php?itemid=106

avatar brianteeman
brianteeman - comment - 19 Dec 2020

the $headerLink needs to be something like
$headerLink = 'index.php?option=com_content&view=article&id=' . $articleIds[10] . '&catid=' . $catIds[4];
but I couldnt get the value of the catid. Must be missing something obvious.

avatar HLeithner HLeithner - change - 19 Dec 2020
Title
[4.0] Blog Sampledata: Fix links, use camelCaps for variable catIds
[4.0] Blog Sampledata: Fix links, use camelCase for variable catIds
avatar HLeithner HLeithner - edited - 19 Dec 2020
avatar laoneo laoneo - change - 22 Dec 2020
Labels Added: ?
avatar chmst
chmst - comment - 27 Jan 2021

@brianteeman changed as desired. The categoryIds were missing in step 3 ...


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

avatar joomla-cms-bot joomla-cms-bot - change - 7 Feb 2021
Category Front End Plugins Administration Language & Strings Front End Plugins
avatar chmst chmst - change - 7 Feb 2021
The description was changed
avatar chmst chmst - edited - 7 Feb 2021
avatar chmst chmst - change - 7 Feb 2021
The description was changed
avatar chmst chmst - edited - 7 Feb 2021
avatar chmst chmst - change - 7 Feb 2021
Labels Added: ?
avatar chmst chmst - change - 10 Feb 2021
The description was changed
avatar chmst chmst - edited - 10 Feb 2021
avatar chmst chmst - change - 10 Feb 2021
The description was changed
avatar chmst chmst - edited - 10 Feb 2021
avatar richard67 richard67 - test_item - 12 Feb 2021 - Tested successfully
avatar richard67
richard67 - comment - 12 Feb 2021

I have tested this item successfully on 42362d8


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

avatar richard67
richard67 - comment - 12 Feb 2021

@brianteeman I'd be happy if you could find some time and test @chmst 's PR here. Thanks in advance.

avatar infograf768
infograf768 - comment - 13 Feb 2021

Title missing for Special Menu Module

Screen Shot 2021-02-13 at 18 27 00

avatar chmst chmst - change - 3 Mar 2021
The description was changed
avatar chmst chmst - edited - 3 Mar 2021
avatar chmst
chmst - comment - 3 Mar 2021

Use access levels for changing login / logout links in top menu and footer menu #32556

avatar chmst chmst - change - 3 Mar 2021
The description was changed
avatar chmst chmst - edited - 3 Mar 2021
avatar chmst chmst - change - 3 Mar 2021
The description was changed
avatar chmst chmst - edited - 3 Mar 2021
avatar richard67
richard67 - comment - 3 Mar 2021

@chmst Does your last change close issue #32556 ?

avatar chmst
chmst - comment - 3 Mar 2021

@infograf768 this was intended as example for a module without title.

avatar chmst
chmst - comment - 6 Mar 2021

Don't activate workflow automatically. #32387

avatar chmst chmst - change - 6 Mar 2021
The description was changed
avatar chmst chmst - edited - 6 Mar 2021
avatar chmst chmst - change - 6 Mar 2021
The description was changed
avatar chmst chmst - edited - 6 Mar 2021
avatar chmst chmst - change - 13 Mar 2021
The description was changed
avatar chmst chmst - edited - 13 Mar 2021
avatar chmst chmst - change - 13 Mar 2021
The description was changed
avatar chmst chmst - edited - 13 Mar 2021
avatar chmst chmst - change - 13 Mar 2021
The description was changed
avatar chmst chmst - edited - 13 Mar 2021
avatar chmst chmst - change - 13 Mar 2021
The description was changed
avatar chmst chmst - edited - 13 Mar 2021
avatar sksuryan
sksuryan - comment - 13 Mar 2021

While trying to login, facing a 404.

image

Steps to reproduce:

  • Go the landing page.
  • Click on login tab.
  • Use the login form on the right.

image

Side notes: The same issue doesn't seem to happen with the form on the left. The form on the left redirects to home page.

Details about system:
I'm currently on Windows 10, using WAMP server. I can confirm this issue for both latest Mozilla Firefox and Microsoft Edge.

I used a fresh installation of Joomla 4.0.8 Nightly with a new database. Next, I installed the patch tester and applied the patch. Lastly, I installed the Sample Blog Data.

The expected behavior is that the user is redirected to either home page or profile page.

avatar chmst
chmst - comment - 14 Mar 2021

@sksuryan thank you for testing. I have repeated your test and it is only if the active view is the login view, right?

avatar sksuryan
sksuryan - comment - 15 Mar 2021

Yes.
Also, I'm a little confused. I don't understand why do we need two login forms? If I'm not wrong, the form on the right is the part of the standard sidebar which we see on every page, but it kind of looks redundant here.

avatar drmenzelit
drmenzelit - comment - 15 Mar 2021

Yes.
Also, I'm a little confused. I don't understand why do we need two login forms? If I'm not wrong, the form on the right is the part of the standard sidebar which we see on every page, but it kind of looks redundant here.

You don't need to have the two... the one is a module, the other is the view of a component. You can use whatever suit better for you, but in sample data we have the two alternatives to show what is possible.

avatar infograf768
infograf768 - comment - 15 Mar 2021

If I'm not wrong, the form on the right is the part of the standard sidebar which we see on every page, but it kind of looks redundant here.

That login module may be published or not and, if published, not on every page.
Also, when one is using, for example, a readmore which needs user to be logged, it is the login menu item form which is presented to the user to fill, and not the module. ;)

avatar sksuryan
sksuryan - comment - 15 Mar 2021

Yes.
Also, I'm a little confused. I don't understand why do we need two login forms? If I'm not wrong, the form on the right is the part of the standard sidebar which we see on every page, but it kind of looks redundant here.

You don't need to have the two... the one is a module, the other is the view of a component. You can use whatever suit better for you, but in sample data we have the two alternatives to show what is possible.

If I'm not wrong, the form on the right is the part of the standard sidebar which we see on every page, but it kind of looks redundant here.

That login module may be published or not and, if published, not on every page.
Also, when one is using, for example, a readmore which needs user to be logged, it is the login menu item form which is presented to the user to fill, and not the module. ;)

okay, yes that actually makes a lot of sense. thank you!

avatar chmst
chmst - comment - 15 Mar 2021

It is confusing.
When Joomla is installed, a main menu (home) and a login module are stored. This is independent of blog sample data. I don't know if there is a reason in history why the login module is added during installation. As @infograf768 says, login and logout is necessary, but if it is needed for installaition?

Blog Sample Data are optional and should show how modules can be added to front-and backend, and especially how Login and Logout can be changed using user groups.

avatar brianteeman
brianteeman - comment - 15 Mar 2021

I don't know if there is a reason in history why the login module is added during installation.

No good reason

avatar chmst
chmst - comment - 20 Mar 2021

To resolve the #31711 (comment) the login module is now hidden on login view.

avatar chmst chmst - change - 20 Mar 2021
The description was changed
avatar chmst chmst - edited - 20 Mar 2021
avatar srishty-07 srishty-07 - test_item - 29 Mar 2021 - Tested successfully
avatar srishty-07
srishty-07 - comment - 29 Mar 2021

I have tested this item successfully on 88293c3

Tested it successfully on Joomla! 4.0.0-beta8-dev ,php 8.0.0


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

avatar sandramay0905 sandramay0905 - test_item - 4 Apr 2021 - Tested successfully
avatar sandramay0905
sandramay0905 - comment - 4 Apr 2021

I have tested this item successfully on 88293c3


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

avatar richard67 richard67 - change - 4 Apr 2021
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 4 Apr 2021

RTC


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

avatar chmst chmst - change - 4 Apr 2021
Status Ready to Commit Pending
avatar chmst
chmst - comment - 4 Apr 2021

Back to pending - the admin menu looks strange and needs new tests.


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

avatar chmst chmst - change - 4 Apr 2021
The description was changed
avatar chmst chmst - edited - 4 Apr 2021
avatar sandramay0905
sandramay0905 - comment - 5 Apr 2021

@chmst Some Questions about "Actual result BEFORE applying this Pull Request"

Backend:

  • There is a menuItem anf a module "Author menu"

Found only menu "Admin Menu", module has title "Author menu".

  • There are some menuItems where the link contains "id=id="

Found this menus, are there more?

  1. About your home page, id=119
  2. Millions, id=121
  3. Love, id=122
  4. Typography, id=106

Frontend:

  • If the user logs out, a 404 occurs

How to get a 404, #31711 (comment) don't bring 404 up.
Test Login and -out at Menu Home and Module Login Form didn't get the 404. Maybe solved elsewhere?

Sorry for wrong test, i test only issue #32991.

avatar brianteeman brianteeman - test_item - 15 Apr 2021 - Tested successfully
avatar brianteeman
brianteeman - comment - 15 Apr 2021

I have tested this item successfully on 4d3ac23


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

avatar sandramay0905
sandramay0905 - comment - 16 Apr 2021

@chmst can you please answer #31711 (comment) so i can do a test?

avatar sksuryan sksuryan - test_item - 16 Apr 2021 - Tested successfully
avatar sksuryan
sksuryan - comment - 16 Apr 2021

I have tested this item successfully on 4d3ac23


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

avatar richard67 richard67 - change - 16 Apr 2021
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 16 Apr 2021

RTC


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

avatar richard67
richard67 - comment - 16 Apr 2021

RTC


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

avatar chmst
chmst - comment - 16 Apr 2021

@sandramay0905 thank you very much for testing and sorry for delay.

Found only menu "Admin Menu", module has title "Author menu".

Correct. The name of the module must not be the same as the link. The title of the module is not shown in the frontend, this is intended.

Found this menus, are there more?

There were 4 menuItems - but which IDs they have depends on your installation. A user can work in an application before adding the sample data and then IDs are different.

Test Login and -out at Menu Home and Module Login Form didn't get the 404. Maybe solved elsewhere?

I also cannot reproduce this effect with 404. It leads me now to the profile page (which is nearly as bad as 404). So probably one of the recent PRs has fixed this.

avatar rdeutz rdeutz - change - 17 Apr 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-04-17 09:55:46
Closed_By rdeutz
Labels Added: ?
avatar rdeutz rdeutz - close - 17 Apr 2021
avatar rdeutz rdeutz - merge - 17 Apr 2021

Add a Comment

Login with GitHub to post a comment