View | Details | Raw Unified | Return to bug 29924
Collapse All | Expand All

(-)a/api/v1/swagger/definitions/patron.yaml (+7 lines)
Lines 328-333 properties: Link Here
328
    description: patron's extended attributes
328
    description: patron's extended attributes
329
    items:
329
    items:
330
      $ref: patron_extended_attribute.yaml
330
      $ref: patron_extended_attribute.yaml
331
  password_expiration_date:
332
    type:
333
      - string
334
      - "null"
335
    format: date
336
    readOnly: true
337
    description: day a patron's password expires and must be changed/reset
331
additionalProperties: false
338
additionalProperties: false
332
required:
339
required:
333
  - surname
340
  - surname
(-)a/api/v1/swagger/paths/patrons.yaml (+5 lines)
Lines 189-194 Link Here
189
        description: Case insensitive search on expiry_date
189
        description: Case insensitive search on expiry_date
190
        required: false
190
        required: false
191
        type: string
191
        type: string
192
      - name: password_expiration_date
193
        in: query
194
        description: Case insensitive search on password_expiration_date
195
        required: false
196
        type: string
192
      - name: incorrect_address
197
      - name: incorrect_address
193
        in: query
198
        in: query
194
        description: Search on incorrect_address
199
        description: Search on incorrect_address
(-)a/t/db_dependent/api/v1/patrons.t (-1 / +5 lines)
Lines 342-347 subtest 'add() tests' => sub { Link Here
342
        delete $newpatron->{patron_id};
342
        delete $newpatron->{patron_id};
343
        delete $newpatron->{restricted};
343
        delete $newpatron->{restricted};
344
        delete $newpatron->{anonymized};
344
        delete $newpatron->{anonymized};
345
        delete $newpatron->{password_expiration_date};
345
346
346
        # Create a library just to make sure its ID doesn't exist on the DB
347
        # Create a library just to make sure its ID doesn't exist on the DB
347
        my $library_to_delete = $builder->build_object({ class => 'Koha::Libraries' });
348
        my $library_to_delete = $builder->build_object({ class => 'Koha::Libraries' });
Lines 395-400 subtest 'add() tests' => sub { Link Here
395
        delete $newpatron->{patron_id};
396
        delete $newpatron->{patron_id};
396
        delete $newpatron->{restricted};
397
        delete $newpatron->{restricted};
397
        delete $newpatron->{anonymized};
398
        delete $newpatron->{anonymized};
399
        delete $newpatron->{password_expiration_date};
398
        $patron_to_delete->delete;
400
        $patron_to_delete->delete;
399
401
400
        # Set a date field
402
        # Set a date field
Lines 620-625 subtest 'update() tests' => sub { Link Here
620
        delete $newpatron->{patron_id};
622
        delete $newpatron->{patron_id};
621
        delete $newpatron->{restricted};
623
        delete $newpatron->{restricted};
622
        delete $newpatron->{anonymized};
624
        delete $newpatron->{anonymized};
625
        delete $newpatron->{password_expiration_date};
623
626
624
        $t->put_ok("//$userid:$password@/api/v1/patrons/-1" => json => $newpatron)
627
        $t->put_ok("//$userid:$password@/api/v1/patrons/-1" => json => $newpatron)
625
          ->status_is(404)
628
          ->status_is(404)
Lines 698-703 subtest 'update() tests' => sub { Link Here
698
        $newpatron->{patron_id} = $unauthorized_patron->to_api->{patron_id};
701
        $newpatron->{patron_id} = $unauthorized_patron->to_api->{patron_id};
699
        $newpatron->{restricted} = $unauthorized_patron->to_api->{restricted};
702
        $newpatron->{restricted} = $unauthorized_patron->to_api->{restricted};
700
        $newpatron->{anonymized} = $unauthorized_patron->to_api->{anonymized};
703
        $newpatron->{anonymized} = $unauthorized_patron->to_api->{anonymized};
704
        $newpatron->{password_expiration_date} = $unauthorized_patron->to_api->{password_expiration_date};
701
705
702
        my $got = $result->tx->res->json;
706
        my $got = $result->tx->res->json;
703
        my $updated_on_got = delete $got->{updated_on};
707
        my $updated_on_got = delete $got->{updated_on};
Lines 729-734 subtest 'update() tests' => sub { Link Here
729
        delete $newpatron->{patron_id};
733
        delete $newpatron->{patron_id};
730
        delete $newpatron->{restricted};
734
        delete $newpatron->{restricted};
731
        delete $newpatron->{anonymized};
735
        delete $newpatron->{anonymized};
736
        delete $newpatron->{password_expiration_date};
732
737
733
        # attempt to update
738
        # attempt to update
734
        $authorized_patron->flags( 2**4 )->store; # borrowers flag = 4
739
        $authorized_patron->flags( 2**4 )->store; # borrowers flag = 4
735
- 

Return to bug 29924