Replaced all getRootLogger() with getLogger(this.getClass())
This commit is contained in:
parent
2198cefdec
commit
b0cd57a750
src/main/java/se/su/dsv/scipro
data/controllers/impl
json
DefaultUserFullLookup.javaDefaultUserLookup.javaImportWorkerLookup.javaJsonResponseHandler.javaRequestSender.java
peer
security/auth
workerthreads
@ -188,11 +188,11 @@ public class FinalSeminarUploadControllerImpl implements FinalSeminarUploadContr
|
||||
}
|
||||
|
||||
} catch (IOException e) {
|
||||
Logger.getRootLogger().log(Level.ERROR,
|
||||
Logger.getLogger(this.getClass()).log(Level.ERROR,
|
||||
"Error while storing seminardocument: " + e.getMessage());
|
||||
throw new Exception(e);
|
||||
} catch (PersistenceException e) {
|
||||
Logger.getRootLogger().log(Level.ERROR,
|
||||
Logger.getLogger(this.getClass()).log(Level.ERROR,
|
||||
"Error while storing seminardocument : " + e.getMessage());
|
||||
if (fd != null) {
|
||||
fileDescriptionDao.delete(fd);
|
||||
@ -233,11 +233,11 @@ public class FinalSeminarUploadControllerImpl implements FinalSeminarUploadContr
|
||||
SupervisorProjectsFinalSeminarPage.getAbsoluteURL());
|
||||
}
|
||||
} catch (IOException e) {
|
||||
Logger.getRootLogger().log(Level.ERROR,
|
||||
Logger.getLogger(this.getClass()).log(Level.ERROR,
|
||||
"Error while storing opposition report: " + e.getMessage());
|
||||
throw new Exception(e);
|
||||
} catch (PersistenceException e) {
|
||||
Logger.getRootLogger().log(Level.ERROR,
|
||||
Logger.getLogger(this.getClass()).log(Level.ERROR,
|
||||
"Error while storing opposition: " + e.getMessage());
|
||||
|
||||
if (fd != null) {
|
||||
|
@ -111,7 +111,7 @@ public class HandInControllerImpl implements HandInController, IClusterable {
|
||||
return true;
|
||||
} catch (Exception e) {
|
||||
//Something went wrong...
|
||||
Logger.getRootLogger().log(Level.ERROR, "An error occured while storing a handin..."+ e.getMessage());
|
||||
Logger.getLogger(this.getClass()).log(Level.ERROR, "An error occured while storing a handin..."+ e.getMessage());
|
||||
|
||||
//Delete from repository
|
||||
//TODO: Possible delete the hand in activity folder for the event as well if it's empty
|
||||
|
@ -81,7 +81,7 @@ public class HandInFeedbackControllerImpl implements HandInFeedbackController {
|
||||
handIn.setStatus(oldStatus);
|
||||
handInFeedbackDao.save(handIn.getFeedback());
|
||||
handInDao.save(handIn);
|
||||
Logger.getRootLogger().log(Level.ERROR, "Failed to store feedback " + e.getMessage());
|
||||
Logger.getLogger(this.getClass()).log(Level.ERROR, "Failed to store feedback " + e.getMessage());
|
||||
return false;
|
||||
}
|
||||
}
|
||||
|
@ -40,7 +40,7 @@ public class DefaultUserFullLookup implements IUserLookup {
|
||||
private Logger logger;
|
||||
|
||||
public DefaultUserFullLookup(){
|
||||
logger = Logger.getRootLogger();
|
||||
logger = Logger.getLogger(this.getClass());
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -39,7 +39,7 @@ public class DefaultUserLookup implements IUserLookup {
|
||||
private Logger logger;
|
||||
|
||||
public DefaultUserLookup(){
|
||||
logger = Logger.getRootLogger();
|
||||
logger = Logger.getLogger(this.getClass());
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -33,7 +33,7 @@ public class ImportWorkerLookup {
|
||||
private Logger logger;
|
||||
|
||||
public ImportWorkerLookup(){
|
||||
logger = Logger.getRootLogger();
|
||||
logger = Logger.getLogger(this.getClass());
|
||||
}
|
||||
|
||||
public void lookup(Date updatedAfter, String type) throws IOException{
|
||||
|
@ -71,7 +71,7 @@ public abstract class JsonResponseHandler implements IResponseHandler {
|
||||
|
||||
public JsonResponseHandler(){
|
||||
super();
|
||||
logger = Logger.getRootLogger();
|
||||
logger = Logger.getLogger(this.getClass());
|
||||
}
|
||||
|
||||
|
||||
|
@ -52,7 +52,7 @@ public class RequestSender {
|
||||
|
||||
public RequestSender(IResponseHandler responseHandler, String url, Map<String, String> parameters, int requestType){
|
||||
|
||||
logger = Logger.getRootLogger();
|
||||
logger = Logger.getLogger(this.getClass());
|
||||
|
||||
if(requestType != REQUEST_TYPE_GET && requestType != REQUEST_TYPE_POST){
|
||||
logger.log(Level.ERROR, "Could not send request, no request type specified");
|
||||
|
@ -237,7 +237,7 @@ public class PeerPortalControllerImpl implements PeerPortalController {
|
||||
|
||||
return review;
|
||||
} catch (Exception e) {
|
||||
Logger.getRootLogger().log(Level.ERROR,
|
||||
Logger.getLogger(this.getClass()).log(Level.ERROR,
|
||||
"Error while accepting peer request: " + e.getMessage());
|
||||
if (review.getId() != null) {
|
||||
// Something must have gone wrong while updating request status
|
||||
@ -286,7 +286,7 @@ public class PeerPortalControllerImpl implements PeerPortalController {
|
||||
return peerRequestDao.save(request);
|
||||
}
|
||||
} catch (Exception e) {
|
||||
Logger.getRootLogger().log(Level.ERROR,
|
||||
Logger.getLogger(this.getClass()).log(Level.ERROR,
|
||||
"Error while storing Peer Request: " + e.getMessage());
|
||||
if (request.getId() != null) {
|
||||
peerRequestDao.delete(request);
|
||||
@ -328,7 +328,7 @@ public class PeerPortalControllerImpl implements PeerPortalController {
|
||||
review.setFile(fdesc.get(0));
|
||||
return true;
|
||||
} catch (Exception e) {
|
||||
Logger.getRootLogger().log(Level.ERROR,
|
||||
Logger.getLogger(this.getClass()).log(Level.ERROR,
|
||||
"Error while storing file for peer review: " + e.getMessage());
|
||||
if (review.getFile() != null) {
|
||||
// Clean up the repository
|
||||
|
@ -43,7 +43,7 @@ public class ExpiredReviewResetWorker extends AbstractWorker {
|
||||
Date expirationDate = new DateTime().minusDays(settings.getNumDaysToSubmitPeerReview()).toDate();
|
||||
|
||||
if( peerReview.getDateCreated().compareTo( expirationDate ) < 0 ) {
|
||||
Logger logger = Logger.getRootLogger();
|
||||
Logger logger = Logger.getLogger(this.getClass());
|
||||
logger.log(Level.INFO,"Expiring peerReview: "+peerReview+" and making "+peerRequest+" available for review again");
|
||||
|
||||
peerReview.setAborted(true);
|
||||
|
@ -39,7 +39,7 @@ public class Authenticator {
|
||||
|
||||
//Catch exceptions if
|
||||
} catch (LoginException le){
|
||||
Logger logger = Logger.getRootLogger();
|
||||
Logger logger = Logger.getLogger(this.getClass());
|
||||
logger.log(Level.ERROR, "System error, Failed to create logincontext:" + le.getMessage());
|
||||
throw le;
|
||||
} catch (SecurityException se){
|
||||
|
@ -105,7 +105,7 @@ public abstract class AbstractWorker extends Thread {
|
||||
this.commitTransaction();
|
||||
} catch ( Exception e ){
|
||||
this.rollbackTransaction();
|
||||
Logger logger = Logger.getRootLogger();
|
||||
Logger logger = Logger.getLogger(this.getClass());
|
||||
logger.log(Level.ERROR, getNameString(this.getClass())+" failed to save it's workerdata!");
|
||||
}
|
||||
|
||||
|
@ -27,7 +27,7 @@ public class UserAndProjectImportWorker extends AbstractWorker{
|
||||
@Override
|
||||
protected void doManualTransactionWork(){
|
||||
Date startDate = this.getStartedWorkingDate();
|
||||
Logger logger = Logger.getRootLogger();
|
||||
Logger logger = Logger.getLogger(this.getClass());
|
||||
logger.log(Level.INFO,"Starting import-from-matchsystem ImportWorker:"+startDate);
|
||||
|
||||
//Get stats of how many Users and Projects changed scince last update
|
||||
|
Loading…
x
Reference in New Issue
Block a user