- Issue created by @ressa
- Status changed to Needs review
over 1 year ago 11:16am 24 September 2023 - last update
over 1 year ago 544 pass - π¦πΉAustria drunken monkey Vienna, Austria
Ah, I see what you mean: our Drush commands output hardly any information, because they use log level βinfoβ which Drush doesnβt display by default. However, for compatibility reasons, we also cannot just use the βsuccessβ log level (which would probably be appropriate in most places). We probably need to switch based on the concrete logger implementation.
Please see/test/review the attached patch.
And thanks for reporting this problem! - π¦πΉAustria drunken monkey Vienna, Austria
Could you please give the patch a try?
- π©π°Denmark ressa Copenhagen
Sure, but I am not sure how to check if it makes any difference in Search API ... How can I check that?
I do still get the Solr error, where it shows "Solr configset for %server_id uploaded", it looks like from this file in the Search API Solr module:
search_api_solr/modules/search_api_solr_admin/src/Commands/SearchApiSolrAdminCommands.php
I think I misunderstood @mkalkbrenner, perhaps an identical issue for Search API Solr should be created for the Solr
%server_id
issue? (I closed π Update README wording for SEARCH_API_SERVER_ID Fixed , since it was about updating the README). -
drunken monkey β
committed 62e3d75b on 8.x-1.x
Issue #3370934 by drunken monkey: Fixed missing log messages in Drush.
-
drunken monkey β
committed 62e3d75b on 8.x-1.x
- Status changed to Fixed
about 1 year ago 8:54am 15 October 2023 - π¦πΉAustria drunken monkey Vienna, Austria
OK, I now see the problem, and created π Fix variable substitution in Drush log messages Needs review in the Solr module queue with the actual problem description and fix.
Still committed the patch above, since it seems to be helpful regardless. - π©π°Denmark ressa Copenhagen
Thanks for fixing the Solr logger string problem in the other issue.
Automatically closed - issue fixed for 2 weeks with no activity.