Fix CP List viewmodels. Bus sadly, newly created CP objects in a list…

… will be saved. Only the second time, when they are modified. This is a major bug
daszat · Jun 28, 2021 · 7795a3f · 7795a3f
1 parent 7d64829
commit 7795a3f
Show file tree

Hide file tree

Showing 5 changed files with 28 additions and 5 deletions.
diff --git a/Zetbox.API/TypeTransformations.cs b/Zetbox.API/TypeTransformations.cs
@@ -215,6 +215,8 @@ private InterfaceType(Type type, IInterfaceTypeChecker typeChecker)
             typeof(IRelationEntry),
             typeof(IValueCollectionEntry<,>),
             typeof(IValueCollectionEntry),
+            typeof(IValueListEntry<,>),
+            typeof(IValueListEntry),
             typeof(IPersistenceObject) 
         };
 

diff --git a/Zetbox.API/Utils/ReadOnlyObservableProjection.cs b/Zetbox.API/Utils/ReadOnlyObservableProjection.cs
@@ -104,7 +104,8 @@ protected virtual void OnCollectionChanged(NotifyCollectionChangedEventArgs args
         {
             if (CollectionChanged != null)
             {
-                CollectionChanged(this, args);
+                // TODO: CollectionChanged(this, args);
+                CollectionChanged(this, new NotifyCollectionChangedEventArgs(NotifyCollectionChangedAction.Reset));
             }
         }
 

diff --git a/Zetbox.Assets/Zetbox.Assets.csproj b/Zetbox.Assets/Zetbox.Assets.csproj
@@ -132,6 +132,7 @@
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Base.BaseParameter.x-zb-Pseudo.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Base.Blob.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Base.Blob.x-zb-Pseudo.resx" />
+    <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Base.CompoundObject.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Base.Constraint.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Base.Constraint.x-zb-Pseudo.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Base.DataType.resx" />
@@ -148,8 +149,10 @@
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Base.Group.x-zb-Pseudo.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Base.ICustomFulltextFormat.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Base.ICustomFulltextFormat.x-zb-Pseudo.resx" />
+    <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Base.Identity.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Base.IInvocation.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Base.IInvocation.x-zb-Pseudo.resx" />
+    <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Base.IMergeable.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Base.INamedObject.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Base.INamedObject.x-zb-Pseudo.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Base.InstanceConstraint.resx" />
@@ -198,12 +201,15 @@
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.GUI.Template.x-zb-Pseudo.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.GUI.ViewModelDescriptor.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.GUI.ViewModelDescriptor.x-zb-Pseudo.resx" />
+    <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.LicenseManagement.License.resx" />
+    <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.LicenseManagement.PrivateKey.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Projekte.Auftrag.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Projekte.Auftrag.x-zb-Pseudo.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Projekte.Mitarbeiter.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Projekte.Mitarbeiter.x-zb-Pseudo.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Projekte.Projekt.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Projekte.Projekt.x-zb-Pseudo.resx" />
+    <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.Projekte.Task.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.SchemaMigration.MigrationProject.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.SchemaMigration.MigrationProject.x-zb-Pseudo.resx" />
     <EmbeddedResource Include="ZetboxBase\Methods\Zetbox.App.SchemaMigration.SourceColumn.resx" />
@@ -410,6 +416,8 @@
     <EmbeddedResource Include="ZetboxBase\Properties\Zetbox.App.GUI.Visual.x-zb-Pseudo.resx" />
     <EmbeddedResource Include="ZetboxBase\Properties\Zetbox.App.GUI.YearFilterConfiguration.resx" />
     <EmbeddedResource Include="ZetboxBase\Properties\Zetbox.App.GUI.YearFilterConfiguration.x-zb-Pseudo.resx" />
+    <EmbeddedResource Include="ZetboxBase\Properties\Zetbox.App.LicenseManagement.License.resx" />
+    <EmbeddedResource Include="ZetboxBase\Properties\Zetbox.App.LicenseManagement.PrivateKey.resx" />
     <EmbeddedResource Include="ZetboxBase\Properties\Zetbox.App.Projekte.Auftrag.resx" />
     <EmbeddedResource Include="ZetboxBase\Properties\Zetbox.App.Projekte.Auftrag.x-zb-Pseudo.resx" />
     <EmbeddedResource Include="ZetboxBase\Properties\Zetbox.App.Projekte.Kunde.resx" />
@@ -496,6 +504,7 @@
     <EmbeddedResource Include="ZetboxBase\Properties\Zetbox.App.Test.SecurityTestChild.x-zb-Pseudo.resx" />
     <EmbeddedResource Include="ZetboxBase\Properties\Zetbox.App.Test.SecurityTestParent.resx" />
     <EmbeddedResource Include="ZetboxBase\Properties\Zetbox.App.Test.SecurityTestParent.x-zb-Pseudo.resx" />
+    <EmbeddedResource Include="ZetboxBase\Properties\Zetbox.App.Test.TestCompany.resx" />
     <EmbeddedResource Include="ZetboxBase\Properties\Zetbox.App.Test.TestCustomObject.resx" />
     <EmbeddedResource Include="ZetboxBase\Properties\Zetbox.App.Test.TestCustomObject.x-zb-Pseudo.resx" />
     <EmbeddedResource Include="ZetboxBase\Properties\Zetbox.App.Test.TestDecimal.resx" />

diff --git a/Zetbox.Client/Models/ColumnDisplayModel.cs b/Zetbox.Client/Models/ColumnDisplayModel.cs
@@ -539,9 +539,13 @@ private void BuildColumns(Zetbox.App.Base.DataType cls, IEnumerable<Property> pr
             if (props == null) throw new ArgumentNullException("props");
             if (methods == null) throw new ArgumentNullException("methods");
 
-            ShowIcon = cls.ShowIconInLists;
-            ShowId = cls.ShowIdInLists;
-            ShowName = cls.ShowNameInLists;
+            if (cls is ObjectClass)
+            {
+                // Makes no sense on CompoundObjects 
+                ShowIcon = cls.ShowIconInLists;
+                ShowId = cls.ShowIdInLists;
+                ShowName = cls.ShowNameInLists;
+            }
 
             this.Columns = new ObservableCollection<ColumnDisplayModel>(
                 props.Select(p => ColumnDisplayModel.Create(mode, p))

diff --git a/Zetbox.Client/Models/PropertyValueModel.cs b/Zetbox.Client/Models/PropertyValueModel.cs
@@ -101,7 +101,14 @@ public static IValueModel GetPropertyValueModel(this Property prop, INotifyingOb
                 var cop = (CompoundObjectProperty)prop;
                 if (cop.IsList)
                 {
-                    return new CompoundCollectionPropertyValueModel(obj, cop);
+                    if (cop.HasPersistentOrder)
+                    {
+                        return new CompoundListPropertyValueModel(obj, cop);
+                    }
+                    else
+                    {
+                        return new CompoundCollectionPropertyValueModel(obj, cop);
+                    }
                 }
                 else
                 {