chore: make lint happy

.github/workflows/lint-php.yml

@@ -16,7 +16,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        php-versions: ["7.4", "8.0", "8.1"]
+        php-versions: ["8.1"]
 
     name: php-lint
 

.github/workflows/phpunit-oci.yml

@@ -20,7 +20,7 @@ jobs:
 
     strategy:
       matrix:
-        php-versions: ["8.0"]
+        php-versions: ["8.3"]
         server-versions: ["master"]
 
     services:

.reuse/dep5

@@ -7,10 +7,18 @@ Files: package-lock.json package.json composer.json composer.lock src/.prettierr
 Copyright: Magnus Anderssen <[email protected]>
 License: AGPL-3.0-or-later
 
-Files: l10n/*.js l10n/*.json bin/tools/translationtool.phar
+Files: bin/tools/translationtool.phar
 Copyright: Nextcloud translators <https://www.transifex.com/nextcloud/>
 License: AGPL-3.0-or-later
 
+Files: l10n/*.js l10n/*.json
+Copyright: Magnus Anderssen <[email protected]>
+License: AGPL-3.0-or-later
+
+Files:*.svelte *.ts
+Copyright: Magnus Anderssen <[email protected]>
+License: AGPL-3.0-or-later
+
 Files: .github/* bin/*.sh translationfiles/*/*.po translationfiles/templates/thesearchpage.pot
 Copyright: Magnus Anderssen <[email protected]>
 License: AGPL-3.0-or-later

lib/Controller/PageController.php

@@ -21,9 +21,11 @@ class PageController extends Controller
     private $labels;
 
 
-    public function __construct(IRequest $request, private IL10N $l, private IInitialState $state,
-    )
-    {
+    public function __construct(
+        IRequest $request,
+        private IL10N $l,
+        private IInitialState $state,
+    ) {
         parent::__construct(Application::APP_ID, $request);
 
         $this->labels = [
@@ -39,7 +41,7 @@ public function __construct(IRequest $request, private IL10N $l, private IInitia
             "Clear current query",
             "Search",
             "Click to change providers",
-            "Filters",            
+            "Filters",
         ];
     }
 

src/.prettierrc

@@ -3,9 +3,20 @@
 	"singleQuote": true,
 	"trailingComma": "none",
 	"printWidth": 100,
-	"plugins": ["prettier-plugin-svelte"],
-	"pluginSearchDirs": ["."],
+	"plugins": [
+		"prettier-plugin-svelte"
+	],
+	"pluginSearchDirs": [
+		"."
+	],
 	"svelteBracketNewLine": false,
 	"bracketSameLine": true,
-	"overrides": [{ "files": "*.svelte", "options": { "parser": "svelte" } }]
-}
+	"overrides": [
+		{
+			"files": "*.svelte",
+			"options": {
+				"parser": "svelte"
+			}
+		}
+	]
+}

src/components/Header/ProviderSelector.svelte

@@ -1,5 +1,6 @@
 <script lang="ts">
 	import { createEventDispatcher } from 'svelte';
+	import type { ChangeEventHandler } from 'svelte/elements';
 	import { writable } from 'svelte/store';
 	import { PROVIDER_ALL, PROVIDER_ALL_LABEL } from '../../lib/search';
 	import { _t } from '../../lib/translate';
@@ -28,14 +29,14 @@
 		}
 	});
 
-	function updateAll(event: InputEvent) {
+	const updateAll: ChangeEventHandler<HTMLInputElement> = (event) => {
 		const target = event.target as HTMLInputElement;
 		if (target.checked) {
 			displayedSelection.set(providers.map(({ id }) => id));
 		} else {
 			displayedSelection.set([]);
 		}
-	}
+	};
 </script>
 
 <div class="mwb-checkboxes-container">

src/lib/log.ts

@@ -1,15 +1,15 @@
 /**
  * Warn with a magnifying glass
- * @param rest 
+ * @param rest
  */
 export function cwarn(...rest: unknown[]) {
-    console.warn('🔎 SEARCH_PAGE::', ...rest)
+	console.warn('🔎 SEARCH_PAGE::', ...rest);
 }
 
 /**
  * Log with a magnifying glass
- * @param rest 
+ * @param rest
  */
 export function clog(...rest: unknown[]) {
-    console.log('🔎 SEARCH_PAGE::', ...rest)
-}
+	console.log('🔎 SEARCH_PAGE::', ...rest);
+}

src/lib/session.ts

@@ -9,9 +9,9 @@ export function saveInSession(terms: string, providerIds: string[]) {
 		newState.append('terms', terms);
 	}
 	if (providerIds) {
-		providerIds.forEach(p => {
+		providerIds.forEach((p) => {
 			newState.append('provider', p);
-		})
+		});
 	}
 
 	try {

src/lib/translate.ts

@@ -3,14 +3,16 @@
 
 import { get } from 'svelte/store';
 import { labels } from '../states/initialStates';
-import { cwarn } from './log'
+import { cwarn } from './log';
 
 export function _t(label: string) {
 	const translations = get(labels);
 	if (translations[label]) {
 		return translations[label];
 	}
 
-	cwarn(`Missing label "${label}" in used language: please help translate: https://github.com/callmemagnus/nextcloud-searchpage`);
+	cwarn(
+		`Missing label "${label}" in used language: please help translate: https://github.com/callmemagnus/nextcloud-searchpage`
+	);
 	return label;
 }

src/states/query.ts

@@ -8,7 +8,7 @@ const session = loadFromSession();
 
 export const terms = writable(session.terms);
 export const providerId = writable(session.providers[0]);
-export const providerIds = writable(session.providers)
+export const providerIds = writable(session.providers);
 
 terms.subscribe((t) => {
 	saveInSession(t.trim(), [get(providerId)]);
@@ -17,5 +17,5 @@ providerId.subscribe((p) => {
 	saveInSession(get(terms), [p]);
 });
 providerIds.subscribe((pIds) => {
-	saveInSession(get(terms), pIds)
-})
+	saveInSession(get(terms), pIds);
+});