ERP-2141: CustomValueModels #18

Merged
merged 9 commits into from Apr 13, 2012

Projects

None yet

2 participants

@cnadeau
Contributor
cnadeau commented Apr 3, 2012

Refactor pour rendre accessible les maps et ajouter le UndoRedoManager dans DefaultPresentationModel

Christian Na... added some commits Mar 29, 2012
Christian Nadeau ERP-2141: Refactor du DynamicPresentationModel pour ajouter une factory
et permettre de dériver dans Framework pour les besoins spécifiques au
tables Swing.
9334e9e
Christian Nadeau ERP-2141: Re-déplacer l'abonnement au value changed dans
registerValueModel.
f4f7b18
Christian Nadeau ERP-2141: Permettre de passer le UndoRedoManager au
DefaultPresentationModel pour permettre le undo/redo pour les
CustomValueModel.
52c194c
@ebelanger ebelanger commented on the diff Apr 4, 2012
src/com/netappsid/binding/DynamicPresentationModel.java
@@ -18,7 +18,7 @@
@SuppressWarnings("serial")
public class DynamicPresentationModel extends PresentationModel
@ebelanger
ebelanger Apr 4, 2012 Contributor

l'encapsulation de cette classe semble totalement brisé avec les nouvelles modifications. Est-ce vraiment nécessaire de rendre autant de méthodes protected? Je ne vois pas de modification dans le framework qui prennent avantage de cette dé-encapsulation. Peut-être que cette classe devrait déléger une partie de son traitement à une nouvelle classe et qui pourra 'manager' cette ouverture de comportement sans briser l'encapsulation de celle-ci?

Christian Na... added some commits Apr 4, 2012
Christian Nadeau ERP-2141: Refactor pour extérioriser la création des ValueModel et les
listeners pour le Map bean change et Map key/value change pour que
binding et framework puisse avoir 2 behaviors distinct pour injecter les
CustomValueModel.
654e5af
Christian Nadeau ERP-2141: Mini refactor pour améliorer la performance ae41ef2
@ebelanger ebelanger commented on an outdated diff Apr 11, 2012
...om/netappsid/binding/DefaultMapBeanChangeHandler.java
@@ -0,0 +1,42 @@
+package com.netappsid.binding;
+
+import java.beans.PropertyChangeEvent;
+import java.util.Map;
+import java.util.Map.Entry;
+
+import com.jgoodies.binding.value.ValueModel;
+
+class DefaultMapBeanChangeHandler implements DynamicPresentationModelMapBeanChangeHandler
@ebelanger
ebelanger Apr 11, 2012 Contributor

pas de tests unitaires sur cette classe.

@ebelanger
Contributor

aucun tests unitaires dans ce pull request.

@cnadeau
Contributor
cnadeau commented Apr 11, 2012

@ebelanger je suis lazy, my bad, tests ajoutés :-p

@cnadeau
Contributor
cnadeau commented Apr 13, 2012

ajout d'un test sur la factory

@ebelanger ebelanger commented on an outdated diff Apr 13, 2012
.../binding/DynamicPresentationModelFactoryImplTest.java
+import com.netappsid.binding.DefaultPresentationModelTest.TestSubModel1;
+import com.netappsid.binding.beans.support.StandardChangeSupportFactory;
+import com.netappsid.observable.StandardObservableCollectionSupportFactory;
+
+public class DynamicPresentationModelFactoryImplTest
+{
+ private PresentationModel rootModel;
+
+ @Before
+ public void setUp()
+ {
+ rootModel = new DefaultPresentationModel(new StandardChangeSupportFactory(), new StandardObservableCollectionSupportFactory(), TestSubModel1.class);
+ }
+
+ @Test
+ public void test()
@ebelanger
ebelanger Apr 13, 2012 Contributor

nom significatif.

@ebelanger ebelanger commented on an outdated diff Apr 13, 2012
.../binding/DynamicPresentationModelFactoryImplTest.java
+{
+ private PresentationModel rootModel;
+
+ @Before
+ public void setUp()
+ {
+ rootModel = new DefaultPresentationModel(new StandardChangeSupportFactory(), new StandardObservableCollectionSupportFactory(), TestSubModel1.class);
+ }
+
+ @Test
+ public void test()
+ {
+ DynamicPresentationModelFactoryImpl factory = new DynamicPresentationModelFactoryImpl();
+ DynamicPresentationModel newDynamicPresentationModel = factory.newDynamicPresentationModel(rootModel, "property1");
+
+ assertTrue(newDynamicPresentationModel.getValueModelFactory() instanceof DynamicPresentationModelValueModelFactoryImpl);
@ebelanger
ebelanger Apr 13, 2012 Contributor

split des asserts qui ne sont pas reliés ensemble.

@ebelanger ebelanger commented on the diff Apr 13, 2012
...psid/binding/DynamicPresentationModelFactoryImpl.java
@@ -0,0 +1,19 @@
+package com.netappsid.binding;
+
+import java.util.Map;
+
+public class DynamicPresentationModelFactoryImpl implements DynamicPresentationModelFactory
+{
+ @Override
+ public DynamicPresentationModel newDynamicPresentationModel(PresentationModel parentModel, String propertyName)
+ {
+ DynamicPresentationModel dynamicPresentationModel = new DynamicPresentationModel(parentModel.getChangeSupportFactory(),
@ebelanger
ebelanger Apr 13, 2012 Contributor

tester les intéraction avec le parent model.

@cnadeau
Contributor
cnadeau commented Apr 13, 2012

done

@ebelanger ebelanger merged commit 221c46f into 360-Innovations:branch_v2.4 Apr 13, 2012
@ebelanger
Contributor

done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment