J4 Issue
avatar brianteeman
brianteeman
31 Aug 2019

Multiple issues

1. on install there are 5 messages in the database but the counter only says 3 and the component only displays 3

2. neither the link in the status bar use the eid for joomla

3. In the database change the eid for one of the messages that is displayed to that of an extension you have installed. you will now see that extension listed in the select box but if you select that extension the message is not displayed or the newsfeed removed. the content doesnt change at all but the url changes from /index.php?option=com_postinstall to index.php

The remaining issue is with sites that have been updated. When you click reset messages you will get all the messages from 3.x which is absolutely fine except that we have language strings missing. (Ignoring the newer messages which I assume have not been merged into j4 yet).

Of these messages I guess we can put the strings back into j4 (or modified versions of them) as they are still important information for users who have upgraded.

The only one that this doesnt apply to is the pre-update check. Should we just delete this record on update?
image

avatar brianteeman brianteeman - open - 31 Aug 2019
avatar joomla-cms-bot joomla-cms-bot - change - 31 Aug 2019
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 31 Aug 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 1 Sep 2019
Status New Discussion
avatar franz-wohlkoenig franz-wohlkoenig - change - 1 Sep 2019
Labels Added: J4 Issue
avatar franz-wohlkoenig franz-wohlkoenig - labeled - 1 Sep 2019
avatar Quy
Quy - comment - 24 Sep 2019

Do we really need the dropdown items? Entries are not added frequently that require it on each page load. Personally, it will be one of the modules to disable.

avatar Quy Quy - change - 30 Dec 2019
Labels Added: ?
avatar Quy Quy - labeled - 30 Dec 2019
avatar coolcat-creations
coolcat-creations - comment - 12 Apr 2020

Do we really need the dropdown items? Entries are not added frequently that require it on each page load. Personally, it will be one of the modules to disable.

+1

avatar SharkyKZ
SharkyKZ - comment - 29 Apr 2020

on install there are 5 messages in the database but the counter only says 3 and the component only displays 3

This seems intentional. Postinstall scripts have some code for hiding the messages when not needed. E.g. HTTP Headers message is not shown when the plugin is enabled.

avatar chmst
chmst - comment - 18 Jun 2020

Please test #29702

avatar Formatio-hippocampi
Formatio-hippocampi - comment - 25 Sep 2020

looks solved: image

avatar brianteeman
brianteeman - comment - 25 Sep 2020

Nope all the reported issues are still present

avatar bembelimen
bembelimen - comment - 16 Oct 2020

Multiple issues

1. on install there are 5 messages in the database but the counter only says 3 and the component only displays 3

So I went through all of the messages and checked the condition:
id 1 is displayed if NO tfa is activated
id 2 is displayed all the time
id 3 is displayed all the time (there is a condition check, which always returns true)
id 4 is displayed if the update check plugin is activated + the caching time in the installer options is set to 0
id 5 is displayed if the httpheaders plugin is not enabled

  1. neither the link in the status bar use the eid for joomla

  2. In the database change the eid for one of the messages that is displayed to that of an extension you have installed. you will now see that extension listed in the select box but if you select that extension the message is not displayed or the newsfeed removed. the content doesnt change at all but the url changes from /index.php?option=com_postinstall to index.php

The eid is used for filtering the list. So you could set a model state to get post install messages from other extensions (fallback is the joomla core extension (files_joomla)). But the populatestate method is missing I guess, which pushs the request parameter to the model.

avatar brianteeman
brianteeman - comment - 18 Oct 2020

@bembelimen thanks for the check and for the PR. The only issue now is with upgrades from 3.x - I will update the original post with more details

avatar brianteeman brianteeman - change - 18 Oct 2020
The description was changed
avatar brianteeman brianteeman - edited - 18 Oct 2020
avatar alikon
alikon - comment - 24 Oct 2020

for me the better thing to do is to simply delete that messages on update to 4

avatar chmst chmst - change - 28 Feb 2021
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2021-02-28 13:48:32
Closed_By chmst
Labels Added: ?
Removed: ?
avatar chmst chmst - close - 28 Feb 2021
avatar chmst
chmst - comment - 28 Feb 2021

With PR #31838 all issues should be resolved, so I am closing this.

avatar chmst chmst - change - 28 Feb 2021
Labels Removed: ? ?
avatar chmst chmst - unlabeled - 28 Feb 2021
avatar chmst chmst - unlabeled - 28 Feb 2021

Add a Comment

Login with GitHub to post a comment