/*
 * Copyright 2002-2005 the original author or authors.
 * 
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 * 
 *      http://www.apache.org/licenses/LICENSE-2.0
 * 
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

package org.springframework.beans.factory.config;

import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;

import org.springframework.beans.factory.FactoryBean;

/**
 * Factory bean for
 * <a href="http://jakarta.apache.org/commons/logging.html">commons-logging</a>
 * Log instances.
 * Will expose the created Log object on getBean calls, and can be passed
 * to bean properites of type org.apache.commons.logging.Log.
 *
 * <p>Useful for sharing Log instances among multiple beans instead of using
 * one Log instance per class name, e.g. for common log topics.
 *
 * @author Juergen Hoeller
 * @since 16.11.2003
 * @see org.apache.commons.logging.Log
 */
public class CommonsLogFactoryBean implements FactoryBean {

  private Log log = null;

  public void setLogName(String logName) {
		if (logName == null) {
			throw new IllegalArgumentException("'logName' must be specified");
		}
    this.log = LogFactory.getLog(logName);
  }

  public Object getObject() {
    return log;
  }

	public Class getObjectType() {
		return (this.log != null ? this.log.getClass() : Log.class);
	}

  public boolean isSingleton() {
    return true;
  }

}

@@ -808,16 +808,75 @@
 oryBean;
+%0Aimport org.springframework.beans.factory.InitializingBean;
 %0A%0A/**%0A *
@@ -1036,15 +1036,28 @@
  on 
+%3Ccode%3E
 getBean
+%3C/code%3E
  cal
@@ -1074,18 +1074,18 @@
 n be
+%0A *
  passed
-%0A *
  to 
@@ -1095,18 +1095,18 @@
 n proper
-i
 t
+i
 es of ty
@@ -1108,16 +1108,22 @@
 of type 
+%3Ccode%3E
 org.apac
@@ -1144,16 +1144,23 @@
 ging.Log
+%3C/code%3E
 .%0A *%0A * 
@@ -1448,16 +1448,34 @@
 toryBean
+, InitializingBean
  %7B%0A%0A  pr
@@ -1491,15 +1491,8 @@
  log
- = null
 ;%0A%0A 
@@ -1537,21 +1537,106 @@
 ) %7B%0A
-%09%09if (logName
+    this.log = LogFactory.getLog(logName);%0A  %7D%0A%0A%09public void afterPropertiesSet() %7B%0A%09%09if (this.log
  == 
@@ -1686,88 +1686,36 @@
 on(%22
-'
 logName
-' must be specified%22);%0A%09%09%7D%0A    this.log = LogFactory.getLog(logName);%0A  
+ is required%22);%0A%09%09%7D%0A%09
 %7D%0A%0A 

check whether logName has been specified in afterPropertiesSet