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

"SoftStart Code \(@softstartcode\) via TikiWiki-cvs" <[email protected]>
Newsgroups gmane.comp.cms.tiki.cvs
Message-ID <68ed8ac248c32_2ce0d44909c@gitlab-sidekiq-low-urgency-cpu-bound-v2-6469d5b86d-tjzfm.mail>

SoftStart Code pushed to branch 27.x at Tiki Wiki CMS Groupware / Tiki


Commits:
f0fa5a0f by Sandeep D at 2025-10-13T23:20:47+00:00
[FIX] Include deprecated feedback type in console verbosity check
---
* [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


(cherry picked from commit 315db4d448ba44f0767d82016f79c5757d3ed1e3)

Co-authored-by: Sandeep D <sandeep.anandbhai-NHSG37FGo5wS/6/[email protected]>

See merge request tikiwiki/tiki!8769

- - - - -


1 changed file:

- lib/core/Feedback.php


Changes:

=====================================
lib/core/Feedback.php
=====================================
@@ -279,7 +279,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/f0fa5a0f59650a55b198547ecdadf5551c87dd24

-- 
View it on GitLab: https://gitlab.com/tikiwiki/tiki/-/commit/f0fa5a0f59650a55b198547ecdadf5551c87dd24
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.