QRunIO/qqq
2
0
Fork 0
mirror of https://github.com/Kingsrook/qqq.git synced 2025-07-17 20:50:44 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove usages of log4j's placeholders-based log formatting calls (some of which were accidental and wrong anyway)

Browse Source
This commit is contained in:
darin.kelkhoff
2024-04-09 18:52:43 -05:00
parent c32d9110ce
commit 3440d45060
2 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
qqq-backend-module-filesystem/src/main/java/com/kingsrook/qqq/backend/module/filesystem/local/actions/AbstractFilesystemAction.java
View File

@ -49,6 +49,7 @@ import com.kingsrook.qqq.backend.module.filesystem.base.model.metadata.AbstractF
import com.kingsrook.qqq.backend.module.filesystem.base.utils.SharedFilesystemBackendModuleUtils; import com.kingsrook.qqq.backend.module.filesystem.base.utils.SharedFilesystemBackendModuleUtils;
import com.kingsrook.qqq.backend.module.filesystem.exceptions.FilesystemException; import com.kingsrook.qqq.backend.module.filesystem.exceptions.FilesystemException;
import org.apache.commons.io.FileUtils; import org.apache.commons.io.FileUtils;
import static com.kingsrook.qqq.backend.core.logging.LogUtils.logPair;
/******************************************************************************* /*******************************************************************************
@ -183,7 +184,7 @@ public class AbstractFilesystemAction extends AbstractBaseFilesystemAction<File>
// if the file doesn't exist, just exit with noop. don't throw an error - that should only // // if the file doesn't exist, just exit with noop. don't throw an error - that should only //
// happen if the "contract" of the method is broken, and the file still exists // // happen if the "contract" of the method is broken, and the file still exists //
////////////////////////////////////////////////////////////////////////////////////////////// //////////////////////////////////////////////////////////////////////////////////////////////
LOG.debug("Not deleting file [{}], because it does not exist.", file); LOG.debug("Not deleting file, because it does not exist.", logPair("file", file));
return; return;
} }
@ -218,7 +219,7 @@ public class AbstractFilesystemAction extends AbstractBaseFilesystemAction<File>
////////////////////////////////////////////////////////////////////////////////////// //////////////////////////////////////////////////////////////////////////////////////
if(!destinationParent.exists()) if(!destinationParent.exists())
{ {
LOG.debug("Making destination directory {} for move", destinationParent.getAbsolutePath()); LOG.debug("Making destination directory for move", logPair("directory", destinationParent.getAbsolutePath()));
if(!destinationParent.mkdirs()) if(!destinationParent.mkdirs())
{ {
throw (new FilesystemException("Failed to make destination directory " + destinationParent.getAbsolutePath() + " to move " + source + " into.")); throw (new FilesystemException("Failed to make destination directory " + destinationParent.getAbsolutePath() + " to move " + source + " into."));

4
qqq-middleware-slack/src/main/java/com/kingsrook/qqq/slack/QSlackImplementation.java
View File

@ -662,11 +662,11 @@ public class QSlackImplementation
////////////////////////////////////////////////////////////////////////// //////////////////////////////////////////////////////////////////////////
// Print result, which includes information about the message (like TS) // // Print result, which includes information about the message (like TS) //
////////////////////////////////////////////////////////////////////////// //////////////////////////////////////////////////////////////////////////
LOG.info("Slack post result {}", result); LOG.info("Slack post result: " + result);
} }
catch(IOException | SlackApiException e) catch(IOException | SlackApiException e)
{ {
LOG.error("error: {}", e.getMessage(), e); LOG.error("error", e);
} }
} }