[PATCH yew-widget-toolkit 2/5] widget: data table: update listeners when store or selection change

[Dominik Csapak <d.csapak@proxmox.com>]

The listeners for the store and the selection are only registered when
the table is created. If the parent passes a different store or
selection later, the table still listens to the old ones, so changes
to the new store or selection don't update the table until something
else triggers a render.

To reproduce:
* render a data table with a store
* re-render the parent with a new store
* change the data of the new store, e.g. with 'set_data'

The table still shows the data the new store had when it was passed
in.

Fix this by registering the listeners again when the store or the
selection changes, like we already do in the SelectionView, and update
the selection status for a new selection.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
 src/widget/data_table/data_table.rs | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/widget/data_table/data_table.rs b/src/widget/data_table/data_table.rs
index b7a3941..6a30dc2 100644
--- a/src/widget/data_table/data_table.rs
+++ b/src/widget/data_table/data_table.rs
@@ -1813,10 +1813,19 @@ impl<S: DataStore + 'static> Component for PwtDataTable<S> {
         let props = ctx.props();
 
         if props.store != old_props.store {
-            // store changed
+            self._store_observer = props
+                .store
+                .add_listener(ctx.link().callback(|_| Msg::DataChange));
             self.update_scroll_info(props);
         }
 
+        if props.selection != old_props.selection {
+            self._selection_observer = props.selection.as_ref().map(|selection| {
+                selection.add_listener(ctx.link().callback(|_| Msg::SelectionChange))
+            });
+            self.update_selection_status(props);
+        }
+
         if props.headers != old_props.headers {
             self.init_headers(props);
         }
-- 
2.47.3