No Code Attached Yet
avatar brianteeman
brianteeman
27 Jul 2023

I dont know the convention for capitalisation on filenames but this looks to be wrong

plugins\schemaorg\blogposting\src\Extension\Blogposting.php
plugins\schemaorg\jobposting\src\Extension\JobPosting.php

If it is wrong then the namespace statements will need to be updated as well to match (and maybe elsewhere)

avatar brianteeman brianteeman - open - 27 Jul 2023
avatar joomla-cms-bot joomla-cms-bot - change - 27 Jul 2023
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 27 Jul 2023
avatar brianteeman
brianteeman - comment - 27 Jul 2023

@richard67 if the filename is changed does there need to be something done to remove the installed file on case sensitive os

avatar richard67
richard67 - comment - 27 Jul 2023

@richard67 if the filename is changed does there need to be something done to remove the installed file on case sensitive os

@brianteeman See here: https://github.com/joomla/joomla-cms/blob/5.0-dev/administrator/components/com_admin/script.php#L2311-L2319 . The tool which I use to generate/update the lists of deleted files and folders will also generate/update this list of renamed files, so I will handle that when time has come.

avatar brianteeman
brianteeman - comment - 27 Jul 2023

Thanks @richard67 now lets see if anyone answers if the filename is right or wrong

avatar bembelimen
bembelimen - comment - 27 Jul 2023

Filename is wrong, you're right. Fixed it here

avatar brianteeman
brianteeman - comment - 27 Jul 2023

Needs to be fixed the way @richard67 explained

avatar HLeithner HLeithner - change - 27 Jul 2023
Status New Closed
Closed_Date 0000-00-00 00:00:00 2023-07-27 19:00:04
Closed_By HLeithner
avatar HLeithner HLeithner - close - 27 Jul 2023
avatar HLeithner
HLeithner - comment - 27 Jul 2023

we are in alpha, we have no upgrade promise between alpha so no it's not needed to do it like described by @richard67

I'm closing this since we have a PR #41260

avatar richard67
richard67 - comment - 27 Jul 2023

we are in alpha, we have no upgrade promise between alpha so no it's not needed to do it like described by @richard67

@HLeithner That does not mean that we should break updates just because we can. Anyway, I will add that one line when I will make the deleted files and so on the next time.

avatar HLeithner
HLeithner - comment - 27 Jul 2023

it's use less to have this rename for all version upgrades till 5.4 in the update script.

avatar richard67
richard67 - comment - 27 Jul 2023

it's use less to have this rename for all version upgrades till 5.4 in the update script.

@HLeithner The same applies for all the other renames in the list. So you want real updates for alphas just to save a few nanoseconds on each update? Maybe we should remove that function for renaming completely and just let things break like it was before we added that function.

So what I take from this is that I should stop to maintain the lists of deleted files and folders and renamed files in the 5.0-dev branch because it was obviously useless all the time.

avatar brianteeman
brianteeman - comment - 27 Jul 2023

In the distant past we had a policy NOT to support updates during development cycle but that changed. I forget when. Clearly that decision was made because we have been providing sql and file updates

avatar brianteeman
brianteeman - comment - 27 Jul 2023

Might as well stop building the nightly updates as well

Add a Comment

Login with GitHub to post a comment