I'm not seeing this behavior. Are your snippets representative of your actual code?

Here's the example I whipped up using a simple application-scoped list of POJOS in lieu of an entity. Toggle works, the lists update fine.

ToggleBean.java

package com.example;

import javax.ejb.Remove;
import javax.ejb.Stateful;

import org.jboss.seam.annotations.In;
import org.jboss.seam.annotations.Name;

@Name("toggle")
@Stateful
public class ToggleBean implements Toggle {
    @In
    Table selectedTable;

    public String toggle() {

        selectedTable.setOn(!selectedTable.isOn());
        return "examplehome";
    }

    @Remove
    public void destroy() {
    }
}

Toggle.java

package com.example;

import javax.ejb.Remove;

public interface Toggle {

    public abstract String toggle();

    @Remove
    public abstract void destroy();

}

TableFactory.java

package com.example;

import java.util.ArrayList;
import java.util.List;

import org.jboss.seam.ScopeType;
import org.jboss.seam.annotations.Factory;
import org.jboss.seam.annotations.Name;

@Name("tableFactory")
public class TableFactory {
    @Factory(value = "allTables", scope = ScopeType.APPLICATION)
    public List<Table> getTables() {
        List<Table> list = new ArrayList<Table>();
        list.add(new Table("a", true));
        list.add(new Table("b", true));
        list.add(new Table("c", false));
        list.add(new Table("d", true));
        list.add(new Table("e", false));
        return list;
    }
}

TableExample.java

package com.example;

import java.util.ArrayList;
import java.util.List;

import javax.ejb.Remove;
import javax.ejb.Stateful;

import org.apache.commons.collections.CollectionUtils;
import org.apache.commons.collections.Predicate;
import org.jboss.seam.ScopeType;
import org.jboss.seam.annotations.Factory;
import org.jboss.seam.annotations.In;
import org.jboss.seam.annotations.Name;
import org.jboss.seam.annotations.Out;
import org.jboss.seam.annotations.datamodel.DataModel;

@Stateful
@Name("pageTest")
public class TableExampleBean implements TableExample {
    @DataModel(scope = ScopeType.PAGE)
    List<Table> list1;
    @DataModel(scope = ScopeType.PAGE)
    List<Table> list2;

    @In(create = true)
    List<Table> allTables;

    @Out(required = false, scope = ScopeType.SESSION)
    Table selectedTable;

    @Factory("list1")
    public void getList1() {
        list1 = new ArrayList<Table>(CollectionUtils.select(allTables,
                new Predicate() {

                    public boolean evaluate(Object arg0) {
                        if (arg0 instanceof Table) {
                            Table table = (Table) arg0;
                            return table.isOn();
                        }

                        return false;
                    }
                }));
    }

    @Factory("list2")
    public void getList2() {
        list2 = new ArrayList<Table>(CollectionUtils.select(allTables,
                new Predicate() {

                    public boolean evaluate(Object arg0) {
                        if (arg0 instanceof Table) {
                            Table table = (Table) arg0;
                            return !table.isOn();
                        }

                        return false;
                    }
                }));
    }

    public String selectTable(Table table) {
        selectedTable = table;
        return "togglePage";
    }

    @Remove
    public void destroy() {
    }
}

TableExample.java

package com.example;

import javax.ejb.Local;

@Local
public interface TableExample {

    public abstract void getList1();

    public abstract void getList2();

    public void destroy();

    public String selectTable(Table table);
}

Table.java

package com.example;

public class Table {
    boolean on;
    String name;

    public Table(String name, boolean on) {
        super();
        this.name = name;
        this.on = on;
    }

    public boolean isOn() {
        return on;
    }

    public void setOn(boolean on) {
        this.on = on;
    }

    public String getName() {
        return name;
    }

    public void setName(String name) {
        this.name = name;
    }
}

lists.page.xml

<!DOCTYPE page PUBLIC "-//JBoss/Seam Pages Configuration DTD 1.1//EN" "http://jboss.com/products/seam/pages-2.0.dtd">

<page>

 <navigation>
  <rule if-outcome="togglePage">
   <redirect view-id="/example/toggle.xhtml" />
  </rule>
 </navigation>

</page>

lists.xhtml

<!DOCTYPE composition PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
                             "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">


<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" xmlns:ui="http://java.sun.com/jsf/facelets"
 xmlns:h="http://java.sun.com/jsf/html" xmlns:f="http://java.sun.com/jsf/core" xmlns:s="http://jboss.com/products/seam/taglib"
 xmlns:a="https://ajax4jsf.dev.java.net/ajax" xmlns:rich="http://richfaces.org/rich">
 <body>

  <h1>Test</h1>

  <div id="messages">
   <h:messages globalOnly="true" styleClass="message" id="globalMessages" />
  </div>

  <h:form>
   <h:dataTable var="table11" value="#{list1}">
    <f:facet name="header">List 1</f:facet>
    <h:column>
     <f:facet name="header">
      <h4></h4>
     </f:facet>
     <h:commandLink action="#{pageTest.selectTable(table11)}">#{table11}</h:commandLink>
    </h:column>
   </h:dataTable>
  </h:form>

  <h:form>
   <h:dataTable var="table12" value="#{list2}">
    <f:facet name="header">List 2</f:facet>
    <h:column>
     <f:facet name="header">
      <h4></h4>
     </f:facet>
     <h:commandLink action="#{pageTest.selectTable(table12)}">#{table12}</h:commandLink>
    </h:column>
   </h:dataTable>
  </h:form>
  <h:form>
   <s:link propagation="false" action="home.xhtml" value="refresh" />
  </h:form>

 </body>
</html>

toggle.page.xml

<!DOCTYPE page PUBLIC "-//JBoss/Seam Pages Configuration DTD 1.1//EN" "http://jboss.com/products/seam/pages-2.0.dtd">

<page>
 <navigation>
  <rule if-outcome="examplehome">
   <redirect view-id="/example/lists.xhtml" />
  </rule>
 </navigation>
</page>

toggle.xhtml

<!DOCTYPE composition PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
                             "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">


<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" xmlns:ui="http://java.sun.com/jsf/facelets"
 xmlns:h="http://java.sun.com/jsf/html" xmlns:f="http://java.sun.com/jsf/core" xmlns:s="http://jboss.com/products/seam/taglib"
 xmlns:a="https://ajax4jsf.dev.java.net/ajax" xmlns:rich="http://richfaces.org/rich">
 <body>

  <h1>Test</h1>

  <div id="messages">
   <h:messages globalOnly="true" styleClass="message" id="globalMessages" />
  </div>



  <h:form>
   #{selectedTable}


   #{selectedTable.on}
   <h:commandButton action="#{toggle.toggle}" />


  </h:form>

 </body>
</html>

I threw this example together in order to illustrate the problem I'm having with DataTables, so using a PickList wouldn't help with that :)

Yes, these code snippets are actual code taken directly from my project.

I'm using JBoss 4.2.2 GA and Seam 2.0

If populateList2 is not getting called, what does the list2 context variable contain after you return from toggle.toggleButton? The old list?

The Factory will only get called if there is no value for the context variable 'list2' - is there a 'list2' defined in a higher scope?

You might try adding a 'org.jboss.seam.postSetVariable.list2' event observer to see when that context variable is getting set.

PS: I'm using Seam 2.0.2-SP1, so that's as recent as I can get right now.

Alain.

Hi Alain,

I didn't have time to spend figuring this out so I abandoned the DataModel and just created a list with getters and setters. It's not elegent and likely adds extra db calls (at least in my case), but it works. My gut feeling on this is that it's probably a SEAM bug, but I don't know how to escalate things to get the dev team to look at it.

Hi,

This is annoying indeed. I guess the developers would love a testcase, but you already pretty much provided that.

The fact that two of us have the same problem is a sign that either:

1) there is a bug

2) there is a user error that is common enough that multiple people fall in the same trap, therefore it deserves the same attention as a bug

Alain.

open a JIRA issue.