The implementation of get/setNewspaperColumns needs to be patched.
It looks like this:

   public int getNewspaperColumns() {
       return _newspaperColumns;
   }
   public void setNewspaperColumns(int newspaperColumns) {
       this._newspaperColumns = newspaperColumns;
   }

It needs to be rewritten to work like get/setRows so that it first
checks a ValueBinding instead of only working with an int.

   public int getRows()
   {
       if (_rows != null)
           return _rows.intValue();
       ValueBinding vb = getValueBinding("rows");
       Number v = vb != null ? (Number) vb.getValue(getFacesContext()) : null;
       return v != null ? v.intValue() : DEFAULT_ROWS;
   }

   public void setRows(int rows)
   {
       _rows = new Integer(rows);
       if (rows < 0)
           throw new IllegalArgumentException("rows: " + rows);
   }

Please open a JIRA issue and attach a patch.  Thanks.



On 11/28/06, Yaron Spektor <[EMAIL PROTECTED]> wrote:
Hi,
I was wondering if anyone knows why the t:dataTable does not accept a
backing bean value for the newspaperColumns value-binding?

Where this example works (4 columns):
<t:dataTable newspaperColumns="4" newspaperOrientation="horizontal"
value="#{bean.valueList}" var="index" >
            <h:column>
                 <h:outputText value="#{index}" />
        </h:column>
</t:dataTable>

But this does not:

<t:dataTable newspaperColumns="#{bean.numberOfColumns}"
newspaperOrientation="horizontal" value="#{bean.valueList}" var="index"
>
            <h:column>
                 <h:outputText value="#{index}" />
        </h:column>
</t:dataTable>

Any work around to that? I can not use the t:columns.

Thanks,

Reply via email to