removing this. to access the static logger property

Author: emacarron

src/main/java/org/mybatis/spring/SqlSessionFactoryBean.java

@@ -354,8 +354,8 @@ protected SqlSessionFactory buildSqlSessionFactory() throws IOException {
       xmlConfigBuilder = new XMLConfigBuilder(this.configLocation.getInputStream(), null, this.configurationProperties);
       configuration = xmlConfigBuilder.getConfiguration();
     } else {
-      if (this.logger.isDebugEnabled()) {
-        this.logger.debug("Property 'configLocation' not specified, using default MyBatis Configuration");
+      if (logger.isDebugEnabled()) {
+        logger.debug("Property 'configLocation' not specified, using default MyBatis Configuration");
       }
       configuration = new Configuration();
       configuration.setVariables(this.configurationProperties);
@@ -375,26 +375,26 @@ protected SqlSessionFactory buildSqlSessionFactory() throws IOException {
       for (String packageToScan : typeAliasPackageArray) {
         configuration.getTypeAliasRegistry().registerAliases(packageToScan,
                 typeAliasesSuperType == null ? Object.class : typeAliasesSuperType);
-        if (this.logger.isDebugEnabled()) {
-          this.logger.debug("Scanned package: '" + packageToScan + "' for aliases");
+        if (logger.isDebugEnabled()) {
+          logger.debug("Scanned package: '" + packageToScan + "' for aliases");
         }
       }
     }
 
     if (!isEmpty(this.typeAliases)) {
       for (Class<?> typeAlias : this.typeAliases) {
         configuration.getTypeAliasRegistry().registerAlias(typeAlias);
-        if (this.logger.isDebugEnabled()) {
-          this.logger.debug("Registered type alias: '" + typeAlias + "'");
+        if (logger.isDebugEnabled()) {
+          logger.debug("Registered type alias: '" + typeAlias + "'");
         }
       }
     }
 
     if (!isEmpty(this.plugins)) {
       for (Interceptor plugin : this.plugins) {
         configuration.addInterceptor(plugin);
-        if (this.logger.isDebugEnabled()) {
-          this.logger.debug("Registered plugin: '" + plugin + "'");
+        if (logger.isDebugEnabled()) {
+          logger.debug("Registered plugin: '" + plugin + "'");
         }
       }
     }
@@ -404,17 +404,17 @@ protected SqlSessionFactory buildSqlSessionFactory() throws IOException {
           ConfigurableApplicationContext.CONFIG_LOCATION_DELIMITERS);
       for (String packageToScan : typeHandlersPackageArray) {
         configuration.getTypeHandlerRegistry().register(packageToScan);
-        if (this.logger.isDebugEnabled()) {
-          this.logger.debug("Scanned package: '" + packageToScan + "' for type handlers");
+        if (logger.isDebugEnabled()) {
+          logger.debug("Scanned package: '" + packageToScan + "' for type handlers");
         }
       }
     }
 
     if (!isEmpty(this.typeHandlers)) {
       for (TypeHandler<?> typeHandler : this.typeHandlers) {
         configuration.getTypeHandlerRegistry().register(typeHandler);
-        if (this.logger.isDebugEnabled()) {
-          this.logger.debug("Registered type handler: '" + typeHandler + "'");
+        if (logger.isDebugEnabled()) {
+          logger.debug("Registered type handler: '" + typeHandler + "'");
         }
       }
     }
@@ -423,8 +423,8 @@ protected SqlSessionFactory buildSqlSessionFactory() throws IOException {
       try {
         xmlConfigBuilder.parse();
 
-        if (this.logger.isDebugEnabled()) {
-          this.logger.debug("Parsed configuration file: '" + this.configLocation + "'");
+        if (logger.isDebugEnabled()) {
+          logger.debug("Parsed configuration file: '" + this.configLocation + "'");
         }
       } catch (Exception ex) {
         throw new NestedIOException("Failed to parse config resource: " + this.configLocation, ex);
@@ -464,13 +464,13 @@ protected SqlSessionFactory buildSqlSessionFactory() throws IOException {
           ErrorContext.instance().reset();
         }
 
-        if (this.logger.isDebugEnabled()) {
-          this.logger.debug("Parsed mapper file: '" + mapperLocation + "'");
+        if (logger.isDebugEnabled()) {
+          logger.debug("Parsed mapper file: '" + mapperLocation + "'");
         }
       }
     } else {
-      if (this.logger.isDebugEnabled()) {
-        this.logger.debug("Property 'mapperLocations' was not specified or no matching resources found");
+      if (logger.isDebugEnabled()) {
+        logger.debug("Property 'mapperLocations' was not specified or no matching resources found");
       }
     }
 

src/main/java/org/mybatis/spring/transaction/SpringManagedTransaction.java

@@ -81,8 +81,8 @@ private void openConnection() throws SQLException {
     this.autoCommit = this.connection.getAutoCommit();
     this.isConnectionTransactional = isConnectionTransactional(this.connection, this.dataSource);
 
-    if (this.logger.isDebugEnabled()) {
-      this.logger.debug(
+    if (logger.isDebugEnabled()) {
+      logger.debug(
           "JDBC Connection ["
               + this.connection
               + "] will"
@@ -96,8 +96,8 @@ private void openConnection() throws SQLException {
    */
   public void commit() throws SQLException {
     if (this.connection != null && !this.isConnectionTransactional && !this.autoCommit) {
-      if (this.logger.isDebugEnabled()) {
-        this.logger.debug("Committing JDBC Connection [" + this.connection + "]");
+      if (logger.isDebugEnabled()) {
+        logger.debug("Committing JDBC Connection [" + this.connection + "]");
       }
       this.connection.commit();
     }
@@ -108,8 +108,8 @@ public void commit() throws SQLException {
    */
   public void rollback() throws SQLException {
     if (this.connection != null && !this.isConnectionTransactional && !this.autoCommit) {
-      if (this.logger.isDebugEnabled()) {
-        this.logger.debug("Rolling back JDBC Connection [" + this.connection + "]");
+      if (logger.isDebugEnabled()) {
+        logger.debug("Rolling back JDBC Connection [" + this.connection + "]");
       }
       this.connection.rollback();
     }