From dc2f447abd4d67638afec26fc7cdb4569d11ff96 Mon Sep 17 00:00:00 2001 From: Stefan Weil Date: Mon, 11 Sep 2023 11:25:59 +0200 Subject: [PATCH 1/2] Fix typos in comments (found by codespell) Signed-off-by: Stefan Weil --- Resources/Public/JavaScript/PageView/PageView.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Resources/Public/JavaScript/PageView/PageView.js b/Resources/Public/JavaScript/PageView/PageView.js index 7c968ef2ef..8f0cbeb171 100644 --- a/Resources/Public/JavaScript/PageView/PageView.js +++ b/Resources/Public/JavaScript/PageView/PageView.js @@ -447,7 +447,7 @@ dlfViewer.prototype.displayHighlightWord = function(highlightWords = null) { this.highlightWords = highlightWords; } - // exctract highlighWords from URL + // extract highlighWords from URL if (this.highlightWords === null) { this.highlightWords = dlfUtils.getUrlParam('tx_dlf[highlight_word]'); } From f54d48e36848774014278b8b4bda2d88d288e593 Mon Sep 17 00:00:00 2001 From: Stefan Weil Date: Sat, 3 Feb 2024 12:07:43 +0100 Subject: [PATCH 2/2] Fix more typos in comments and documentation Most of them were found and fixed using codespell. Signed-off-by: Stefan Weil --- Documentation/Plugins/Index.rst | 2 +- .../JavaScript/jPlayer/jquery.jplayer.js | 26 +++++++++---------- Tests/Functional/Common/MetsDocumentTest.php | 2 +- 3 files changed, 15 insertions(+), 15 deletions(-) diff --git a/Documentation/Plugins/Index.rst b/Documentation/Plugins/Index.rst index 87fcb468a1..cf65955b28 100644 --- a/Documentation/Plugins/Index.rst +++ b/Documentation/Plugins/Index.rst @@ -33,7 +33,7 @@ marker templates for plugins are not supported anymore. Now, all HTML markup is done in Fluid. To use different templates, you have to overload the templates by the common TYPO3 way. -The following TypoScript defines addition paths inside a "example" extenion:: +The following TypoScript defines additional paths inside an "example" extension:: plugin.tx_dlf { view { diff --git a/Resources/Public/JavaScript/jPlayer/jquery.jplayer.js b/Resources/Public/JavaScript/jPlayer/jquery.jplayer.js index 842f31b63d..644b759571 100644 --- a/Resources/Public/JavaScript/jPlayer/jquery.jplayer.js +++ b/Resources/Public/JavaScript/jPlayer/jquery.jplayer.js @@ -699,7 +699,7 @@ playbackRate: 1, // Warning - Now both an option and a status property ended: 0 -/* Persistant status properties created dynamically at _init(): +/* Persistent status properties created dynamically at _init(): width height cssClass @@ -865,7 +865,7 @@ this.options.volume = this._limitValue(this.options.volume, 0, 1); // Limit volume value's bounds. - // Create the formats array, with prority based on the order of the supplied formats string + // Create the formats array, with priority based on the order of the supplied formats string $.each(this.options.supplied.toLowerCase().split(","), function(index1, value1) { var format = value1.replace(/^\s+|\s+$/g, ""); //trim if(self.format[format]) { // Check format is valid. @@ -882,7 +882,7 @@ } }); - // Create the solutions array, with prority based on the order of the solution string + // Create the solutions array, with priority based on the order of the solution string $.each(this.options.solution.toLowerCase().split(","), function(index1, value1) { var solution = value1.replace(/^\s+|\s+$/g, ""); //trim if(self.solution[solution]) { // Check solution is valid. @@ -1212,7 +1212,7 @@ // MJP: The background change remains. Would need to store the original to restore it correctly. // MJP: The jPlayer element's size change remains. - // Clear the media to reset the GUI and stop any downloads. Streams on some browsers had persited. (Chrome) + // Clear the media to reset the GUI and stop any downloads. Streams on some browsers had persisted. (Chrome) this.clearMedia(); // Remove the size/sizeFull cssClass from the cssSelectorAncestor this._removeUiClass(); @@ -1573,7 +1573,7 @@ var ct = 0, cpa = 0, sp = 0, cpr = 0; // Fixes the duration bug in iOS, where the durationchange event occurs when media.duration is not always correct. - // Fixes the initial duration bug in BB OS7, where the media.duration is infinity and displays as NaN:NaN due to Date() using inifity. + // Fixes the initial duration bug in BB OS7, where the media.duration is infinity and displays as NaN:NaN due to Date() using infinity. if(isFinite(media.duration)) { this.status.duration = media.duration; } @@ -2645,18 +2645,18 @@ if(!this.options.fullWindow && this.options[key].cssClass !== value.cssClass) { this._removeUiClass(); } - this.options[key] = $.extend({}, this.options[key], value); // store a merged copy of it, incase not all properties changed. + this.options[key] = $.extend({}, this.options[key], value); // store a merged copy of it, in case not all properties changed. this._refreshSize(); break; case "sizeFull" : if(this.options.fullWindow && this.options[key].cssClass !== value.cssClass) { this._removeUiClass(); } - this.options[key] = $.extend({}, this.options[key], value); // store a merged copy of it, incase not all properties changed. + this.options[key] = $.extend({}, this.options[key], value); // store a merged copy of it, in case not all properties changed. this._refreshSize(); break; case "autohide" : - this.options[key] = $.extend({}, this.options[key], value); // store a merged copy of it, incase not all properties changed. + this.options[key] = $.extend({}, this.options[key], value); // store a merged copy of it, in case not all properties changed. this._updateAutohide(); break; case "loop" : @@ -2670,20 +2670,20 @@ this.options[key] = value; break; case "nativeVideoControls" : - this.options[key] = $.extend({}, this.options[key], value); // store a merged copy of it, incase not all properties changed. + this.options[key] = $.extend({}, this.options[key], value); // store a merged copy of it, in case not all properties changed. this.status.nativeVideoControls = this._uaBlocklist(this.options.nativeVideoControls); this._restrictNativeVideoControls(); this._updateNativeVideoControls(); break; case "noFullWindow" : - this.options[key] = $.extend({}, this.options[key], value); // store a merged copy of it, incase not all properties changed. + this.options[key] = $.extend({}, this.options[key], value); // store a merged copy of it, in case not all properties changed. this.status.nativeVideoControls = this._uaBlocklist(this.options.nativeVideoControls); // Need to check again as noFullWindow can depend on this flag and the restrict() can override it. this.status.noFullWindow = this._uaBlocklist(this.options.noFullWindow); this._restrictNativeVideoControls(); this._updateButtons(); break; case "noVolume" : - this.options[key] = $.extend({}, this.options[key], value); // store a merged copy of it, incase not all properties changed. + this.options[key] = $.extend({}, this.options[key], value); // store a merged copy of it, in case not all properties changed. this.status.noVolume = this._uaBlocklist(this.options.noVolume); this._updateVolume(); this._updateMute(); @@ -2699,7 +2699,7 @@ } break; case "timeFormat" : - this.options[key] = $.extend({}, this.options[key], value); // store a merged copy of it, incase not all properties changed. + this.options[key] = $.extend({}, this.options[key], value); // store a merged copy of it, in case not all properties changed. break; case "keyEnabled" : this.options[key] = value; @@ -2708,7 +2708,7 @@ } break; case "keyBindings" : - this.options[key] = $.extend(true, {}, this.options[key], value); // store a merged DEEP copy of it, incase not all properties changed. + this.options[key] = $.extend(true, {}, this.options[key], value); // store a merged DEEP copy of it, in case not all properties changed. break; case "audioFullScreen" : this.options[key] = value; diff --git a/Tests/Functional/Common/MetsDocumentTest.php b/Tests/Functional/Common/MetsDocumentTest.php index 7e931331a9..49b3b8fa70 100644 --- a/Tests/Functional/Common/MetsDocumentTest.php +++ b/Tests/Functional/Common/MetsDocumentTest.php @@ -152,7 +152,7 @@ public function canGetDownloadLocation() /* * The method `getDownloadLocation` should return a string, but returns null in some cases. - * Therefor, a TypeError must be expected here. + * Therefore, a TypeError must be expected here. */ $this->expectException('TypeError'); $doc->getDownloadLocation('ID_DOES_NOT_EXIST');