[FIX] attachment_preview: pre-commit fixes

- Fix author string concatenation in manifest
- Remove unnecessary eslint-disable-next-line no-undef comments
- Fix trailing whitespace

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: gregory-rutecki

attachment_preview/__manifest__.py

@@ -4,7 +4,7 @@
 {
     "name": "Preview attachments",
     "version": "19.0.1.0.0",
-    "author": "Therp BV," "Onestein," "Odoo Community Association (OCA)",
+    "author": "Therp BV,Onestein,Odoo Community Association (OCA)",
     "website": "https://github.com/OCA/knowledge",
     "license": "AGPL-3",
     "summary": "Preview attachments supported by Viewer.js",

attachment_preview/static/src/js/attachmentPreviewWidget.esm.js

@@ -35,7 +35,7 @@ export class AttachmentPreviewWidget extends Component {
 
     _onPopoutClick() {
         if (!this.attachments[this.state.activeIndex]) return;
-        // eslint-disable-next-line no-undef
+
         window.open(this.attachments[this.state.activeIndex].previewUrl);
     }
 

attachment_preview/static/src/js/mail_core/attachment_list.esm.js

@@ -4,7 +4,7 @@ import {patch} from "@web/core/utils/patch";
 
 patch(AttachmentList.prototype, {
     _onPreviewAttachment(attachment) {
-        // eslint-disable-next-line no-undef
+
         var $target = $(event.currentTarget);
         var split_screen = $target.attr("data-target") !== "new";
         showPreview(

attachment_preview/static/src/js/utils.esm.js

@@ -25,11 +25,11 @@ export function getUrl(
     var url = "";
     if (attachment_url) {
         if (attachment_url.slice(0, 21) === "/web/static/lib/pdfjs") {
-            // eslint-disable-next-line no-undef
+
             url = (window.location.origin || "") + attachment_url;
         } else {
             url =
-                // eslint-disable-next-line no-undef
+
                 (window.location.origin || "") +
                 "/attachment_preview/static/lib/ViewerJS/index.html" +
                 "?type=" +
@@ -38,13 +38,13 @@ export function getUrl(
                 encodeURIComponent(attachment_title) +
                 "&zoom=automatic" +
                 "#" +
-                // eslint-disable-next-line no-undef
+
                 attachment_url.replace(window.location.origin, "");
         }
         return url;
     }
     url =
-        // eslint-disable-next-line no-undef
+
         (window.location.origin || "") +
         "/attachment_preview/static/lib/ViewerJS/index.html" +
         "?type=" +
@@ -74,7 +74,7 @@ export function showPreview(
             attachment_info_list,
         });
     } else {
-        // eslint-disable-next-line no-undef
+
         window.open(
             getUrl(
                 attachment_id,

attachment_preview/static/src/js/viewerjs_tweaks.js

@@ -5,13 +5,13 @@
 // OpenERP's context
 (function (original_Viewer) {
     "use strict";
-    // eslint-disable-next-line no-undef
+
     window.Viewer = function (plugin, parameters) {
         if (!plugin) {
-            // eslint-disable-next-line no-undef, no-alert
+            // eslint-disable-next-line no-alert
             alert("Unsupported file type");
         }
         return original_Viewer(plugin, parameters);
     };
-    // eslint-disable-next-line no-undef
+
 })(window.Viewer);