BUG in com.sun.faces.config.ManagedBeanFactory

Question

While browsing the JSF 1.0 source code, IntelliJ found a "silly assignment" in com.sun.faces.config.ManagedBeanFactory:
    public void setManagedBeanBean(ManagedBeanBean newBean) {
        synchronized (this) {
            //ManagedBeanBean clone method implemented to return deep copy
            this.managedBean = managedBean; // (ManagedBeanBean) newBean.clone();
            //set the scope
            scope = managedBean.getManagedBeanScope();
    }The problem is that the line
this.managedBean = managedBean;is assigning a value to itself, whereas of course it should be:
this.managedBean = newBean;Geoff Rimmer

Answer 1

This has been fixed and will be availabe in the next release.