public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
* [pve-devel] [PATCH widget-toolkit 1/1] fix: utils: correctly append id to task description text
@ 2025-09-16 12:57 Michael Köppl
  2025-09-30 16:07 ` Michael Köppl
  0 siblings, 1 reply; 2+ messages in thread
From: Michael Köppl @ 2025-09-16 12:57 UTC (permalink / raw)
  To: pve-devel

Without this change, only the value of the type variable would be used
as the task description for task types without an entry in the
task_desc_table.

Signed-off-by: Michael Köppl <m.koeppl@proxmox.com>
---
 src/Utils.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Utils.js b/src/Utils.js
index a0c6ca5..236390a 100644
--- a/src/Utils.js
+++ b/src/Utils.js
@@ -770,7 +770,7 @@ Ext.define('Proxmox.Utils', {
             if (!farray) {
                 text = type;
                 if (id) {
-                    type += ' ' + id;
+                    text += ' ' + id;
                 }
                 return text;
             } else if (Ext.isFunction(farray)) {
-- 
2.47.3



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [pve-devel] [PATCH widget-toolkit 1/1] fix: utils: correctly append id to task description text
  2025-09-16 12:57 [pve-devel] [PATCH widget-toolkit 1/1] fix: utils: correctly append id to task description text Michael Köppl
@ 2025-09-30 16:07 ` Michael Köppl
  0 siblings, 0 replies; 2+ messages in thread
From: Michael Köppl @ 2025-09-30 16:07 UTC (permalink / raw)
  To: pve-devel

ping, still applies

On 9/16/25 2:57 PM, Michael Köppl wrote:
> Without this change, only the value of the type variable would be used
> as the task description for task types without an entry in the
> task_desc_table.
> 
> Signed-off-by: Michael Köppl <m.koeppl@proxmox.com>
> ---
>  src/Utils.js | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/Utils.js b/src/Utils.js
> index a0c6ca5..236390a 100644
> --- a/src/Utils.js
> +++ b/src/Utils.js
> @@ -770,7 +770,7 @@ Ext.define('Proxmox.Utils', {
>              if (!farray) {
>                  text = type;
>                  if (id) {
> -                    type += ' ' + id;
> +                    text += ' ' + id;
>                  }
>                  return text;
>              } else if (Ext.isFunction(farray)) {



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2025-09-30 16:07 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-09-16 12:57 [pve-devel] [PATCH widget-toolkit 1/1] fix: utils: correctly append id to task description text Michael Köppl
2025-09-30 16:07 ` Michael Köppl

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal