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