User tests: Successful: Unsuccessful:
Pull Request for Issue #31690 ., #32270
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"
Backend:
Frontend:
Backend:
Frontend:
Status | New | ⇒ | Pending |
Category | ⇒ | Front End Plugins |
Title |
|
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.
Title |
|
Labels |
Added:
?
|
@brianteeman changed as desired. The categoryIds were missing in step 3 ...
Category | Front End Plugins | ⇒ | Administration Language & Strings Front End Plugins |
Labels |
Added:
?
|
I have tested this item
@brianteeman I'd be happy if you could find some time and test @chmst 's PR here. Thanks in advance.
@infograf768 this was intended as example for a module without title.
While trying to login, facing a 404.
Steps to reproduce:
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.
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.
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. ;)
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!
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.
I don't know if there is a reason in history why the login module is added during installation.
No good reason
To resolve the #31711 (comment) the login module is now hidden on login view.
I have tested this item
Tested it successfully on Joomla! 4.0.0-beta8-dev ,php 8.0.0
I have tested this item
Status | Pending | ⇒ | Ready to Commit |
RTC
Status | Ready to Commit | ⇒ | Pending |
Back to pending - the admin menu looks strange and needs new tests.
@chmst Some Questions about "Actual result BEFORE applying this Pull Request"
- 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?
- 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?
I have tested this item
@chmst can you please answer #31711 (comment) so i can do a test?
I have tested this item
Status | Pending | ⇒ | Ready to Commit |
RTC
RTC
@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.
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:
?
|
The menu items are correct now but the typography link is now index.php?itemid=106