?
avatar Scrabble96
Scrabble96
23 Sep 2020

Problem identified

I was looking at PR #30724 where "Dummymethod" has been corrected to "Dummy method" but I noticed that "fullfill" in the same line is incorrect. It should either be "fulfil" (English UK) or "fulfill" (English US).

Additionally, in the Joomla copyright information the word "License" has the US spelling. I was under the impression that Joomla was using UK English for its spelling, so "License" should be "Licence".

This applies to both front and back ends.

Proposed solution

  1. Change "fullfill" to "fulfil" wherever it occurs
  2. Change "License" to "License" wherever it occurs

Open questions

avatar Scrabble96 Scrabble96 - open - 23 Sep 2020
avatar joomla-cms-bot joomla-cms-bot - change - 23 Sep 2020
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 23 Sep 2020
avatar brianteeman
brianteeman - comment - 23 Sep 2020

oops my mistake on fulfil I will fix that now
I dont want to correct the spelling of License as there are external software tools that all assume the american spelling

avatar richard67
richard67 - comment - 23 Sep 2020

Closing as having a pull request for the fulfil change. Please test #30742 .

For the License spelling see the comment above for the reasons not to change.

avatar richard67 richard67 - change - 23 Sep 2020
Status New Closed
Closed_Date 0000-00-00 00:00:00 2020-09-23 10:39:18
Closed_By richard67
avatar richard67 richard67 - close - 23 Sep 2020
avatar richard67
richard67 - comment - 23 Sep 2020

@Scrabble96 Additional info: Of course there are much more occurrences of fulfill than fixed by PR #30742 on an installation, but all these belong to external libraries, and in J4 we don't have them in our sources, so the PR covers all we can do.

avatar Scrabble96
Scrabble96 - comment - 23 Sep 2020

Thank you both @brianteeman and @richard67

Add a Comment

Login with GitHub to post a comment