Notifychanges not working in @listen event

answered 2014-09-22 15:03:06 +0800

chillworld
5367 ● 4 ● 9
https://github.com/chillw...

Hi sathishk,

First of all, @NotifyChange only works for comands and setters, see documentation.

Secondly, If you use @Listen => MVC and @NotifyChange => MVVM you are doing something wrong.

If you need to do a notifychange outside a command or setter you can always use BindUtils:

BindUtils.postNotifyChange(null,null,object,"*");

In mine eye's you have to change the @Listen to a @Command and everything will work.

in zul this shall look like :

<button id="bplanlink" onClick="@command('billingPlan')"/>

Greetz chill.

answered 2014-09-23 13:11:13 +0800

chillworld
5367 ● 4 ● 9
https://github.com/chillw...

updated 2014-09-23 14:56:07 +0800

ZUL :

<zk xmlns:h="native">
    <window id="win" border="none" apply="org.zkoss.bind.BindComposer" viewModel="@id('vm') @init('org.novelerp.controller.DashboardUIController')" xmlns:d="http://www.zkoss.org/2005/zk/client/attribute">
        <div class="dashboard_links">
            <h:ul>
                <h:li class="gu no-border" if="${sessionScope.isCRCAdmin}">
                    <a id="bplanlink" onClick="@command('bplanlink', selected = bplanpanel)">Billing Plan</a>
                </h:li>
            </h:ul>
        </div>
        <tabbox mold="accordion" selectedPanel="@bind(vm.selectedPanel)">
            <tabs>
                <tab sclass="gu" id="tabbplan" style="font-size: 13px;" if="${sessionScope.isCustomer}" visible="@load(vm.tabbplanVisible)">Billing Plan</tab>
            </tabs>
            <tabpanels>
                <tabpanel id="bplanpanel" if="${sessionScope.isCustomer}">
                    <grid model="@bind(vm.billingPlansList2)" >
                        <columns sizable="true">
                            <column label="Billing Date"></column>
                            <column label="Date Description"></column>
                            <column label="MileStone Relavance"></column>
                            <column label="Block"></column>
                            <column label="Billing Status"></column>
                        </columns>
                        <template name="model">
                            <row>
                                <label value="@bind(each.billingdate)"></label>
                                <label value="@bind(each.datedescription)"></label>
                                <label value="@bind(each.milestonerelavance)"></label>
                                <label value="@bind(each.billingblock)"></label>
                                <label value="@bind(each.billingstatus)"></label>
                            </row>
                        </template>
                    </grid>
                </tabpanel>
            </tabpanels>

        </tabbox>
    </window>
</zk>

JAVA

See that class don't extend any class. remove also all the @wire.

public class DashboardUIController {

    private Boolean tabbplanVisible = Boolean.FALSE;
    private Tabpanel selectedPanel;

    public Boolean getTabbplanVisible () {
        return tabbplanVisible;
    }

    public void setTabbplanVisible(Boolean tabbplanVisible) {
        this.tabbplanVisible = tabbplanVisible;
    }

    public Tabpanel getSelectedPanel () {
        return selectedPanel;
    }

    public void setSelectedPanel (Tabpanel selectedPanel) {
        this.selectedPanel = selectedPanel;
    }


    @NotifyChange({"billingPlansList2","selectedPanel","tabbplanVisible"})
    public void billingPlan(@BindingParam("selected") Tabpanel bplanpanel) {
        setSelectedPanel(bplanpanel);
        setTabbplanVisible(Boolean.TRUE);
        billingPlansList2 = new BillingPlanServiceImpl().getBillinPlanList("");

        for (BillingPlan bplan : billingPlansList2) {
            billplan = new BillingPlan();
            System.out.println(":bplan.getBillingdate():-" + bplan.getBillingdate());
            billplan.setBillingdate(bplan.getBillingdate());
        }
    }

}

Try this. Didn't test it(wrote it in notepad++) but it shall be close to what you want.

Greetz chill.

answered 2014-09-25 05:09:02 +0800

hawk
3250 ● 1 ● 5
http://hawkphoenix.blogsp...

updated 2014-09-25 05:32:32 +0800

The root cause of the error of this zkfiddle case, http://zkfiddle.org/sample/3kvld74/23-Tabbox-switching-bug, is that tabbox doesn't allow you to set a null selected tab. Because when you load the page, zk will set vm.selectedTab to tabbox for @bind(vm.selectedTab), for that moment, vm.selectedTab is null.

If you want to control selected tab, you can bind int to "selectedIndex" of a tabbox.

For principle of MVVM pattern, ViewModel should not hold any reference to any component. The principle doesn't encourage users to pass components as parameters to a ViewModel. If you want to change a tabpanel's visibility, you should bind a VM's boolean property to tabpanel "visible" attribute, e.g.

<tabpanel visible="@bind(vm.visible")>

instead of passing that tabpanel to a command method and call tabpanel.setVisible().

answered 2014-09-25 07:28:17 +0800

cor3000
6280 ● 2 ● 7

updated 2014-09-25 07:39:40 +0800

Just to take Tabbox in MVVM a step further:

from ZK7 on to enable adding and removing Tabs dynamically in MVVM you can use a ListModel and template to control the tabs and selection without accessing the components at all.

also check this example on zkfiddle

Robert