Use SLF4J API for logging

doc/release-note

@@ -8,6 +8,7 @@
   Upgrade Notes:
     Update Java compiler source/target to 8
     Update zweb to 9.6.0
+    Use SLF4J API for logging
 
 -----
 1.1.0

zest/classpath

@@ -1,3 +1,4 @@
 ../dist/lib/ext/zcommon.jar
 ../dist/lib/ext/zweb.jar
 ../dist/lib/ext/servlet-api.jar
+../dist/lib/ext/slf4j-api.jar

zest/pom.xml

@@ -61,6 +61,17 @@
 			<version>2.3</version>
 			<scope>provided</scope>
 		</dependency>
+		<dependency>
+			<groupId>org.slf4j</groupId>
+			<artifactId>slf4j-api</artifactId>
+			<version>1.7.5</version>
+		</dependency>
+		<dependency>
+			<groupId>org.slf4j</groupId>
+			<artifactId>slf4j-jdk14</artifactId>
+			<version>1.7.5</version>
+			<scope>runtime</scope>
+		</dependency>
 	</dependencies>
 	<build>
 		<directory>${project.basedir}/debug</directory>

zest/src/org/zkoss/zest/sys/ZestFilter.java

@@ -24,8 +24,10 @@
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 import org.zkoss.lang.Classes;
-import org.zkoss.util.logging.Log;
 
 import org.zkoss.zest.sys.Parser;
 import org.zkoss.zest.sys.impl.ParserImpl;
@@ -40,7 +42,7 @@
  * @author tomyeh
  */
 public class ZestFilter implements Filter {
-	private static final Log log = Log.lookup(ZestManager.class);
+	private static final Logger log = LoggerFactory.getLogger(ZestManager.class);
 
 	private ZestManager _manager;
 
@@ -74,7 +76,7 @@ public final void init(FilterConfig config) throws ServletException {
 		_manager.init(ctx, parser);
 
 		if (oldManager != null)
-			log.warning(oldManager+" is replaced"+_manager);
+			log.warn(oldManager+" is replaced"+_manager);
 	}
 	private static Object newInstance(String clsnm, Class klass) throws ServletException {
 		final Object o;

zest/src/org/zkoss/zest/sys/ZestManager.java

@@ -22,8 +22,10 @@
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 import org.zkoss.lang.reflect.Fields;
-import org.zkoss.util.logging.Log;
 import org.zkoss.web.servlet.http.Https;
 import org.zkoss.web.util.resource.ServletContextLocator;
 
@@ -44,7 +46,7 @@
  * @author tomyeh
  */
 public class ZestManager {
-	private static final Log log = Log.lookup(ZestManager.class);
+	private static final Logger log = LoggerFactory.getLogger(ZestManager.class);
 	private static final String ATTR_MANAGER = "org.zkoss.zest.sys.manager";
 
 	private ServletContext _ctx;

zest/src/org/zkoss/zest/sys/impl/ActionDefinitionImpl.java

@@ -102,7 +102,7 @@ else if (c instanceof String)
 			else
 				throw new ClassNotFoundException("Class expected in "+_klassV+", not "+c);
 		}
-		return klass.newInstance();
+		return klass.getDeclaredConstructor().newInstance();
 	}
 	public String execute(ActionContext ac, Object action) throws Exception {
 		final String mtdnm = (String)_method.getValue(ac);

zest/src/org/zkoss/zest/sys/impl/ParserImpl.java

@@ -21,9 +21,11 @@
 import java.util.HashMap;
 import java.net.URL;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 import org.zkoss.lang.Classes;
 import org.zkoss.util.resource.Locator;
-import org.zkoss.util.logging.Log;
 import org.zkoss.idom.Document;
 import org.zkoss.idom.Element;
 import org.zkoss.idom.Item;
@@ -44,7 +46,7 @@
  * @author tomyeh
  */
 public class ParserImpl implements Parser {
-	private static final Log log = Log.lookup(ParserImpl.class);
+	private static final Logger log = LoggerFactory.getLogger(ParserImpl.class);
 
 	public ParserImpl() {
 	}
@@ -54,7 +56,7 @@ public Configuration parse(URL url, Locator loc) throws Exception {
 		try {
 			return parse(new SAXBuilder(true, false, true).build(url).getRootElement(), loc);
 		} catch (Throwable ex) {
-			log.realCauseBriefly("Failed to parse "+url, ex);
+			log.error("Failed to parse "+url, ex);
 			if (ex instanceof Error) throw (Error)ex;
 			throw (Exception)ex;
 		}