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

(-)a/Koha/Patron.pm (+1 lines)
Lines 1951-1956 sub to_api_mapping { Link Here
1951
        altcontactsurname   => 'altcontact_surname',
1951
        altcontactsurname   => 'altcontact_surname',
1952
        altcontactstate     => 'altcontact_state',
1952
        altcontactstate     => 'altcontact_state',
1953
        altcontactzipcode   => 'altcontact_postal_code',
1953
        altcontactzipcode   => 'altcontact_postal_code',
1954
        password_expiration_date => undef,
1954
        primary_contact_method => undef,
1955
        primary_contact_method => undef,
1955
        secret              => undef,
1956
        secret              => undef,
1956
        auth_method         => undef,
1957
        auth_method         => undef,
(-)a/api/v1/swagger/definitions/patron.yaml (-7 lines)
Lines 359-371 properties: Link Here
359
      - number
359
      - number
360
      - "null"
360
      - "null"
361
    description: Balance of the patron's account
361
    description: Balance of the patron's account
362
  password_expiration_date:
363
    type:
364
      - string
365
      - "null"
366
    format: date
367
    readOnly: true
368
    description: day a patron's password expires and must be changed/reset
369
additionalProperties: false
362
additionalProperties: false
370
required:
363
required:
371
  - surname
364
  - surname
(-)a/api/v1/swagger/paths/patrons.yaml (-5 lines)
Lines 189-199 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
197
      - name: incorrect_address
192
      - name: incorrect_address
198
        in: query
193
        in: query
199
        description: Search on incorrect_address
194
        description: Search on incorrect_address
(-)a/t/db_dependent/api/v1/patrons.t (-6 lines)
Lines 342-348 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};
346
345
347
        # Create a library just to make sure its ID doesn't exist on the DB
346
        # Create a library just to make sure its ID doesn't exist on the DB
348
        my $library_to_delete = $builder->build_object({ class => 'Koha::Libraries' });
347
        my $library_to_delete = $builder->build_object({ class => 'Koha::Libraries' });
Lines 396-402 subtest 'add() tests' => sub { Link Here
396
        delete $newpatron->{patron_id};
395
        delete $newpatron->{patron_id};
397
        delete $newpatron->{restricted};
396
        delete $newpatron->{restricted};
398
        delete $newpatron->{anonymized};
397
        delete $newpatron->{anonymized};
399
        delete $newpatron->{password_expiration_date};
400
        $patron_to_delete->delete;
398
        $patron_to_delete->delete;
401
399
402
        # Set a date field
400
        # Set a date field
Lines 622-628 subtest 'update() tests' => sub { Link Here
622
        delete $newpatron->{patron_id};
620
        delete $newpatron->{patron_id};
623
        delete $newpatron->{restricted};
621
        delete $newpatron->{restricted};
624
        delete $newpatron->{anonymized};
622
        delete $newpatron->{anonymized};
625
        delete $newpatron->{password_expiration_date};
626
623
627
        $t->put_ok("//$userid:$password@/api/v1/patrons/-1" => json => $newpatron)
624
        $t->put_ok("//$userid:$password@/api/v1/patrons/-1" => json => $newpatron)
628
          ->status_is(404)
625
          ->status_is(404)
Lines 701-707 subtest 'update() tests' => sub { Link Here
701
        $newpatron->{patron_id} = $unauthorized_patron->to_api->{patron_id};
698
        $newpatron->{patron_id} = $unauthorized_patron->to_api->{patron_id};
702
        $newpatron->{restricted} = $unauthorized_patron->to_api->{restricted};
699
        $newpatron->{restricted} = $unauthorized_patron->to_api->{restricted};
703
        $newpatron->{anonymized} = $unauthorized_patron->to_api->{anonymized};
700
        $newpatron->{anonymized} = $unauthorized_patron->to_api->{anonymized};
704
        $newpatron->{password_expiration_date} = $unauthorized_patron->to_api->{password_expiration_date};
705
701
706
        my $got = $result->tx->res->json;
702
        my $got = $result->tx->res->json;
707
        my $updated_on_got = delete $got->{updated_on};
703
        my $updated_on_got = delete $got->{updated_on};
Lines 733-739 subtest 'update() tests' => sub { Link Here
733
        delete $newpatron->{patron_id};
729
        delete $newpatron->{patron_id};
734
        delete $newpatron->{restricted};
730
        delete $newpatron->{restricted};
735
        delete $newpatron->{anonymized};
731
        delete $newpatron->{anonymized};
736
        delete $newpatron->{password_expiration_date};
737
732
738
        # attempt to update
733
        # attempt to update
739
        $authorized_patron->flags( 2**4 )->store; # borrowers flag = 4
734
        $authorized_patron->flags( 2**4 )->store; # borrowers flag = 4
740
- 

Return to bug 29924