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 335-340 subtest 'add() tests' => sub { Link Here
335
        delete $newpatron->{patron_id};
335
        delete $newpatron->{patron_id};
336
        delete $newpatron->{restricted};
336
        delete $newpatron->{restricted};
337
        delete $newpatron->{anonymized};
337
        delete $newpatron->{anonymized};
338
        delete $newpatron->{password_expiration_date};
338
339
339
        # Create a library just to make sure its ID doesn't exist on the DB
340
        # Create a library just to make sure its ID doesn't exist on the DB
340
        my $library_to_delete = $builder->build_object({ class => 'Koha::Libraries' });
341
        my $library_to_delete = $builder->build_object({ class => 'Koha::Libraries' });
Lines 388-393 subtest 'add() tests' => sub { Link Here
388
        delete $newpatron->{patron_id};
389
        delete $newpatron->{patron_id};
389
        delete $newpatron->{restricted};
390
        delete $newpatron->{restricted};
390
        delete $newpatron->{anonymized};
391
        delete $newpatron->{anonymized};
392
        delete $newpatron->{password_expiration_date};
391
        $patron_to_delete->delete;
393
        $patron_to_delete->delete;
392
394
393
        # Set a date field
395
        # Set a date field
Lines 576-581 subtest 'update() tests' => sub { Link Here
576
        delete $newpatron->{patron_id};
578
        delete $newpatron->{patron_id};
577
        delete $newpatron->{restricted};
579
        delete $newpatron->{restricted};
578
        delete $newpatron->{anonymized};
580
        delete $newpatron->{anonymized};
581
        delete $newpatron->{password_expiration_date};
579
582
580
        $t->put_ok("//$userid:$password@/api/v1/patrons/-1" => json => $newpatron)
583
        $t->put_ok("//$userid:$password@/api/v1/patrons/-1" => json => $newpatron)
581
          ->status_is(404)
584
          ->status_is(404)
Lines 654-659 subtest 'update() tests' => sub { Link Here
654
        $newpatron->{patron_id} = $unauthorized_patron->to_api->{patron_id};
657
        $newpatron->{patron_id} = $unauthorized_patron->to_api->{patron_id};
655
        $newpatron->{restricted} = $unauthorized_patron->to_api->{restricted};
658
        $newpatron->{restricted} = $unauthorized_patron->to_api->{restricted};
656
        $newpatron->{anonymized} = $unauthorized_patron->to_api->{anonymized};
659
        $newpatron->{anonymized} = $unauthorized_patron->to_api->{anonymized};
660
        $newpatron->{password_expiration_date} = $unauthorized_patron->to_api->{password_expiration_date};
657
661
658
        my $got = $result->tx->res->json;
662
        my $got = $result->tx->res->json;
659
        my $updated_on_got = delete $got->{updated_on};
663
        my $updated_on_got = delete $got->{updated_on};
Lines 685-690 subtest 'update() tests' => sub { Link Here
685
        delete $newpatron->{patron_id};
689
        delete $newpatron->{patron_id};
686
        delete $newpatron->{restricted};
690
        delete $newpatron->{restricted};
687
        delete $newpatron->{anonymized};
691
        delete $newpatron->{anonymized};
692
        delete $newpatron->{password_expiration_date};
688
693
689
        # attempt to update
694
        # attempt to update
690
        $authorized_patron->flags( 2**4 )->store; # borrowers flag = 4
695
        $authorized_patron->flags( 2**4 )->store; # borrowers flag = 4
691
- 

Return to bug 29924