userIdは文字列、idは数字。APIは整数のuser.idを使うのでidを使う。

Author: takuya

src/lib/Backup/BacklogArchiver.php

@@ -36,7 +36,7 @@ public function storable ( BaseModel $obj ): array {
     return $arr;
   }
 
-  protected function filterAttrUser ( $arr, $class ) {
+  protected function filterUserAttrs ( $arr, $class ) {
     $map = $class::attribute_mapping_list();
     $userAttrNames = array_map( fn( $e ) => $e[0], array_filter( $map, fn( $e ) => str_ends_with( $e[1], 'User' ) ) );
     $userArrayAttrNames = array_filter( $userAttrNames, fn( $str ) => $str == StrTool::plural( $str ) );
@@ -46,10 +46,10 @@ protected function filterAttrUser ( $arr, $class ) {
         $arr[$k] = $v['nulabId'] ?? null;
       }
       if ( in_array( $k, $userArrayAttrNames ) ) {
-        $arr[$k] = array_map( fn( $e ) => $e["userId"], $v );
+        $arr[$k] = array_map( fn( $e ) => $e["id"], $v );
       }
       if ( in_array( $k, $userAttrNames ) ) {
-        $arr[$k] = $v["userId"] ?? null;
+        $arr[$k] = $v["id"] ?? null;
       }
     }
 
@@ -69,7 +69,7 @@ protected function filterAttribnutes($arr,$class,$attrName){
     return $arr;
   }
   protected function filterAttr( $arr, $class ) {
-    $arr = $this->filterAttrUser($arr, $class);
+    $arr = $this->filterUserAttrs($arr, $class);
     $arr = $this->filterAttribnutes($arr,$class, 'attachments');
     $arr = $this->filterAttribnutes($arr,$class, 'stars');
     $arr = $this->filterAttribnutes($arr,$class, 'sharedFiles');
@@ -124,7 +124,7 @@ protected function getInterfaceAttrs( $obj ) {
     if( $this->hasInterface($obj, ProjectAttrs::class) ) {
       /** @var ProjectAttrs $obj */
       $cols['teams'] = array_map(fn( $e ) => $e->id, $obj->teams());
-      $cols['users'] = array_map(fn( $e ) => $e->userId, $obj->users());
+      $cols['users'] = array_map(fn( $e ) => $e->id, $obj->users());
     }
 
     return $cols;

tests/Unit/Backup/Storable/IssueStorableTest.php

@@ -25,9 +25,8 @@ protected function assert_issue_is_storable($arr){
     $this->assertIsArrayOfInt($arr['category']);
     $this->assertIsInt($arr['issueType']);
     $this->assertIsInt($arr['status']);
-    $this->assertIsString($arr['createdUser']);
-    $this->assertIsString($arr['assignee']);
-    
+    $this->assertIsInt($arr['createdUser']);
+    $this->assertIsInt($arr['assignee']);
   }
 
   public function test_issue_storable(): void {

tests/Unit/Backup/Storable/ProjectStorableTest.php

@@ -17,7 +17,7 @@ protected function assert_project_is_storable ( $arr ) {
     $names = array_merge( $names, ['icon', 'users', 'teams', 'spaceKey'] );
     //
     $this->assertArrayIsSameValues( $names, array_keys( $arr ) );
-    $this->assertIsArrayOfString( $arr['users'] );
+    $this->assertIsArrayOfInt( $arr['users'] );
     $this->assertNotEmpty( $arr['users'] );
     $this->assertIsArray( $arr['teams'] );
     $this->assertIsArrayOfInt( $arr['teams'] );