buzzn / core

Die Kraft der Gemeinschaft
GNU Affero General Public License v3.0
1 stars 0 forks source link

update instead of create for contact person in organization owner update #1472

Closed scsirdx closed 6 years ago

scsirdx commented 6 years ago
{
   "id":"39",
   "type":"organization",
   "updated_at":"2018-07-31T11:55:49.750Z",
   "name":"Generic organization",
   "phone":"089 / 32 16 8",
   "fax":null,
   "website":"http://ffwww.generic-organization.com",
   "email":"dev+generic-organization@buzzn.net",
   "description":"Description of the generic organization",
   "updatable":true,
   "deletable":false,
   "customer_number":null,
   "bank_accounts":{
      "array":{

      }
   },
   "address":{
      "id":"59",
      "type":"address",
      "updated_at":"2018-07-31T11:29:12.252Z",
      "street":"hkjhklj",
      "city":"hkjlhklj#######",
      "zip":"709879",
      "country":"DE",
      "updatable":true,
      "deletable":true
   },
   "contact":{
      "type":"person",
      "prefix":"M",
      "title":"Dr.",
      "first_name":"Wolfgang",
      "last_name":"Owne###@@@!!",
      "phone":"0926 6198559",
      "fax":"4324324234",
      "email":"dev+owner@buzzn.net",
      "preferred_language":"de",
      "image":"http://localhost:3000/uploads/person/image/41/medium_156005c5baf40ff51a327f1c34f2975b.jpg",
      "updatable":true,
      "bank_accounts":{
         "array":{
            "0":{
               "id":"1",
               "type":"bank_account",
               "updated_at":"2018-07-31T09:59:04.734Z",
               "holder":"Wolfgang Owner",
               "bank_name":"Berliner Volksbank",
               "bic":"BYLADEM1001",
               "iban":"DE89370400440532013000",
               "direct_debit":true,
               "updatable":true,
               "deletable":true
            }
         }
      },
      "address":{
         "id":"36",
         "type":"address",
         "updated_at":"2018-07-31T14:24:40.120Z",
         "street":"Nathaliegasse 95",
         "city":"Karlstadt",
         "zip":"18302",
         "country":"DE",
         "updatable":true,
         "deletable":true
      }
   },
   "legal_representation":{
      "id":"46"
   }
}

Causes 422 with updated_at missing for the contact. It's a PATCH http://localhost:3000/api/admin/localpools/5/organization-owner In this case contact has no id and updatedAt, so should be created as a new person.

mguentner commented 6 years ago

I am not able to reproduce: c4e86d5841f230b69cffd4fa3315bb171d051aba

mguentner commented 6 years ago

fixed in a39b7bf67b5e8389c64129fd71b0809ef56dbb98