J4 Issue ?
avatar brianteeman
brianteeman
12 Sep 2018

While preparing #22141 I see that some components have an sql folder for install and uninstall

I have no idea if these are

  1. needed
  2. up to date

But more importantly some of them only have mysql scripts and no postgres

avatar brianteeman brianteeman - open - 12 Sep 2018
avatar joomla-cms-bot joomla-cms-bot - change - 12 Sep 2018
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 12 Sep 2018
avatar franz-wohlkoenig franz-wohlkoenig - change - 12 Sep 2018
Status New Discussion
avatar franz-wohlkoenig franz-wohlkoenig - change - 12 Sep 2018
Category SQL
avatar alikon
alikon - comment - 12 Sep 2018

for "protected" components they shouldn't be necessary (cause you cannot uninstall them)
for "unistallable" maybe they are needed .....

avatar mbabker
mbabker - comment - 12 Sep 2018

for "unistallable" maybe they are needed .....

Nothing in core is safely uninstallable.

avatar mbabker
mbabker - comment - 12 Sep 2018

I have no idea if these are

  1. needed
  2. up to date

If we actually treated the core extensions as extensions, then yes they would be needed. But as we don't treat the core extensions as extensions, there are a lot of liberties we can take with them, especially in the manifests and whether extension level SQL is maintained.

Personally, I would suggest if it is an extension it is treated as an extension (accurate manifest, local SQL files, etc.) but as we don't manage core extensions as extensions and we definitely don't have the resources to maintain things I'd almost say just strip the core manifests to the minimum needed to not break the extension manager's display and be done with it.

avatar brianteeman
brianteeman - comment - 12 Sep 2018

i have already updated the manifests to match the current file status and it has been merged now

avatar brianteeman brianteeman - change - 12 Sep 2018
Labels Added: J4 Issue
avatar brianteeman brianteeman - labeled - 12 Sep 2018
avatar brianteeman brianteeman - change - 8 Nov 2018
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2018-11-08 13:17:46
Closed_By brianteeman
avatar brianteeman brianteeman - close - 8 Nov 2018

Add a Comment

Login with GitHub to post a comment