fix: fixing syncvar hook not being called in host mode (#918)

MirageNet · Sep 6, 2021 · 7accba7 · 7accba7
1 parent c37c88f
commit 7accba7
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/Assets/Mirage/Weaver/Processors/SyncVarProcessor.cs b/Assets/Mirage/Weaver/Processors/SyncVarProcessor.cs
@@ -98,12 +98,15 @@ bool IsValidSyncVar(FieldDefinition field)
 
         void ProcessSyncVar(FoundSyncVar syncVar)
         {
+            // process attributes first before creating setting, otherwise it wont know about hook
+            syncVar.SetWrapType(module);
+            syncVar.ProcessAttributes();
+
             FieldDefinition fd = syncVar.FieldDefinition;
 
             string originalName = fd.Name;
             Weaver.DebugLog(fd.DeclaringType, $"Sync Var {fd.Name} {fd.FieldType}");
 
-            syncVar.SetWrapType(module);
 
             MethodDefinition get = GenerateSyncVarGetter(syncVar);
             MethodDefinition set = GenerateSyncVarSetter(syncVar);
@@ -126,7 +129,6 @@ void ProcessSyncVar(FoundSyncVar syncVar)
                 propertySiteProcessor.Getters[fd] = get;
             }
 
-            syncVar.ProcessAttributes();
             syncVar.FindSerializeFunctions(writers, readers);
 
             if (syncVar.FloatPackSettings.HasValue)