Merge pull request #3192 from SeedCompany/refactor/owner

Author: CarsonF

dbschema/comments.esdl

@@ -3,7 +3,7 @@ module Comments {
     commentThreads := .<container[is Thread];
   }
 
-  type Thread extending default::Resource, Mixin::Embedded, Mixin::Owned {
+  type Thread extending default::Resource, Mixin::Embedded {
     overloaded required single link container: Aware {
       on target delete delete source;
     };
@@ -12,7 +12,7 @@ module Comments {
     latestComment := (select .comments order by .createdAt desc limit 1);
   }
 
-  type Comment extending default::Resource, Mixin::Owned {
+  type Comment extending default::Resource {
     required thread: Thread {
       on target delete delete source;
     };

dbschema/common.esdl

@@ -1,7 +1,9 @@
 module default {
-  global currentUserId: uuid;
-  alias currentUser := <User>(global currentUserId);
-  
+  global currentActorId: uuid;
+  global currentActor := (select Actor filter .id = global currentActorId);
+  global currentUser := (select User filter .id = global currentActorId);
+  global currentRoles := (global currentActor).roles;
+
   scalar type ReportPeriod extending enum<Monthly, Quarterly>;
 
   # Helper function to workaround native support for sort ignoring accents

dbschema/engagement.esdl

@@ -76,6 +76,8 @@ module default {
       insert Engagement::DedicationCeremony {
         createdAt := datetime_of_statement(),
         modifiedAt := datetime_of_statement(),
+        createdBy := assert_exists(global currentActor),
+        modifiedBy := assert_exists(global currentActor),
         engagement := __new__,
         project := __new__.project,
         projectContext := __new__.projectContext,
@@ -128,6 +130,8 @@ module default {
       insert Engagement::CertificationCeremony {
         createdAt := datetime_of_statement(),
         modifiedAt := datetime_of_statement(),
+        createdBy := assert_exists(global currentActor),
+        modifiedBy := assert_exists(global currentActor),
         engagement := __new__,
         project := __new__.project,
         projectContext := __new__.projectContext,

dbschema/file.esdl

@@ -25,15 +25,9 @@ module File {
     # optional to have tri-state. idk if needed but that's what I implemented in neo4j
     public: bool;
 
-    required createdBy: default::User {
-      default := default::currentUser;
-    };
-    required modifiedBy: default::User {
-      default := default::currentUser;
-      rewrite update using (default::currentUser);
-      # TODO trigger change up the tree
-      # TODO trigger re-eval on node delete?
-    };
+    # TODO trigger change up the tree
+    # TODO trigger re-eval on node delete?
+    # modifiedBy: default::User
 
     # TODO trigger directories evaluate
     required size: int64;