[TikiWiki-commits] [Git][tikiwiki/tiki][master] [FIX] Include deprecated feedback type in console verbosity check

Benoit Grégoire (@benoitg) via TikiWiki-cvs <[email protected]>
Newsgroups gmane.comp.cms.tiki.cvs
Message-ID <68e930352dde_2c9f42dc-15@gitlab-sidekiq-low-urgency-cpu-bound-v2-6c768f58c5-pxh9p.mail>

Benoit Grégoire pushed to branch master at Tiki Wiki CMS Groupware / Tiki


Commits:
315db4d4 by Sandeep D at 2025-10-10T16:03:37+00:00
[FIX] Include deprecated feedback type in console verbosity check
---
* [FIX] Include deprecated feedback type in console verbosity check

Fixes issue where feedback type messages were always printed to console regardless of verbosity level, causing unwanted output in profile application

Problem:
- Feedback::success() sets type to feedback (deprecated but kept for backward compatibility)
- Feedback::printToConsole() only checks for success or note types when determining if verbosity is required
- Result: feedback messages bypass verbosity check and always print
- Affects profile testing: Shows false warnings like in Spreadsheet demo

Solution:
- Added feedback to verbosity check in printToConsole()

See merge request tikiwiki/tiki!8756

- - - - -


1 changed file:

- lib/core/Feedback.php


Changes:

=====================================
lib/core/Feedback.php
=====================================
@@ -302,7 +302,7 @@ class Feedback
                         $message = $type . ': ' . str_replace('<br />', "\n", $message['mes']);
                     }
 
-                    if ($type === 'success' || $type === 'note') {
+                    if ($type === 'success' || $type === 'note' || $type === 'feedback') {
                         if (! $output->isVeryVerbose()) {
                             continue;
                         }



View it on GitLab: https://gitlab.com/tikiwiki/tiki/-/commit/315db4d448ba44f0767d82016f79c5757d3ed1e3

-- 
View it on GitLab: https://gitlab.com/tikiwiki/tiki/-/commit/315db4d448ba44f0767d82016f79c5757d3ed1e3
You're receiving this email because of your account on gitlab.com.

_______________________________________________
TikiWiki-cvs mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/tikiwiki-cvs
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.