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

(-)a/Koha/REST/V1/Acquisitions/Orders.pm (-8 / +2 lines)
Lines 197-211 sub delete { Link Here
197
    }
197
    }
198
198
199
    return try {
199
    return try {
200
201
        $order->delete;
200
        $order->delete;
202
201
        return $c->resource_deleted;
203
        return $c->render(
202
    } catch {
204
            status  => 204,
205
            openapi => q{}
206
        );
207
    }
208
    catch {
209
        $c->unhandled_exception($_);
203
        $c->unhandled_exception($_);
210
    };
204
    };
211
}
205
}
(-)a/Koha/REST/V1/Acquisitions/Vendors.pm (-7 / +2 lines)
Lines 146-158 sub delete { Link Here
146
            unless $vendor;
146
            unless $vendor;
147
147
148
        $vendor->delete;
148
        $vendor->delete;
149
149
        return $c->resource_deleted;
150
        return $c->render(
150
    } catch {
151
            status  => 204,
152
            openapi => q{}
153
        );
154
    }
155
    catch {
156
        $c->unhandled_exception($_);
151
        $c->unhandled_exception($_);
157
    };
152
    };
158
}
153
}
(-)a/Koha/REST/V1/AdvancedEditorMacro.pm (-6 / +4 lines)
Lines 264-272 sub delete { Link Here
264
264
265
    return try {
265
    return try {
266
        $macro->delete;
266
        $macro->delete;
267
        return $c->render( status => 204, openapi => q{} );
267
        return $c->resource_deleted;
268
    }
268
    } catch {
269
    catch {
270
        $c->unhandled_exception($_);
269
        $c->unhandled_exception($_);
271
    };
270
    };
272
}
271
}
Lines 292-300 sub delete_shared { Link Here
292
291
293
    return try {
292
    return try {
294
        $macro->delete;
293
        $macro->delete;
295
        return $c->render( status => 204, openapi => q{} );
294
        return $c->resource_deleted;
296
    }
295
    } catch {
297
    catch {
298
        $c->unhandled_exception($_);
296
        $c->unhandled_exception($_);
299
    };
297
    };
300
}
298
}
(-)a/Koha/REST/V1/ArticleRequests.pm (-10 / +3 lines)
Lines 58-67 sub cancel { Link Here
58
                notes               => $notes
58
                notes               => $notes
59
            }
59
            }
60
        );
60
        );
61
        return $c->render(
61
        return $c->resource_deleted;
62
            status  => 204,
63
            openapi => q{}
64
        );
65
    } catch {
62
    } catch {
66
        $c->unhandled_exception($_);
63
        $c->unhandled_exception($_);
67
    };
64
    };
Lines 99-110 sub patron_cancel { Link Here
99
                notes               => $notes
96
                notes               => $notes
100
            }
97
            }
101
        );
98
        );
102
        return $c->render(
99
        return $c->resource_deleted;
103
            status  => 204,
100
    } catch {
104
            openapi => q{}
105
        );
106
    }
107
    catch {
108
        $c->unhandled_exception($_);
101
        $c->unhandled_exception($_);
109
    };
102
    };
110
}
103
}
(-)a/Koha/REST/V1/Auth/Identity/Provider/Domains.pm (-4 / +1 lines)
Lines 186-195 sub delete { Link Here
186
186
187
    return try {
187
    return try {
188
        $domain->delete;
188
        $domain->delete;
189
        return $c->render(
189
        return $c->resource_deleted;
190
            status  => 204,
191
            openapi => q{}
192
        );
193
    } catch {
190
    } catch {
194
        $c->unhandled_exception($_);
191
        $c->unhandled_exception($_);
195
    };
192
    };
(-)a/Koha/REST/V1/Auth/Identity/Providers.pm (-6 / +2 lines)
Lines 202-213 sub delete { Link Here
202
202
203
    return try {
203
    return try {
204
        $provider->delete;
204
        $provider->delete;
205
        return $c->render(
205
        return $c->resource_deleted;
206
            status  => 204,
206
    } catch {
207
            openapi => q{}
208
        );
209
    }
210
    catch {
211
        $c->unhandled_exception($_);
207
        $c->unhandled_exception($_);
212
    };
208
    };
213
}
209
}
(-)a/Koha/REST/V1/Authorities.pm (-3 / +2 lines)
Lines 112-120 sub delete { Link Here
112
    return try {
112
    return try {
113
        DelAuthority( { authid => $authority->authid } );
113
        DelAuthority( { authid => $authority->authid } );
114
114
115
        return $c->render( status => 204, openapi => q{} );
115
        return $c->resource_deleted;
116
    }
116
    } catch {
117
    catch {
118
        $c->unhandled_exception($_);
117
        $c->unhandled_exception($_);
119
    };
118
    };
120
}
119
}
(-)a/Koha/REST/V1/Biblios.pm (-1 / +1 lines)
Lines 135-141 sub delete { Link Here
135
            );
135
            );
136
        }
136
        }
137
        else {
137
        else {
138
            return $c->render( status => 204, openapi => "" );
138
            return $c->resource_deleted;
139
        }
139
        }
140
    }
140
    }
141
    catch {
141
    catch {
(-)a/Koha/REST/V1/Biblios/ItemGroups.pm (-3 / +2 lines)
Lines 211-219 sub delete { Link Here
211
211
212
    return try {
212
    return try {
213
        $item_group->delete;
213
        $item_group->delete;
214
        return $c->render( status => 204, openapi => '' );
214
        return $c->resource_deleted;
215
    }
215
    } catch {
216
    catch {
217
        $c->unhandled_exception($_);
216
        $c->unhandled_exception($_);
218
    };
217
    };
219
}
218
}
(-)a/Koha/REST/V1/Biblios/ItemGroups/Items.pm (-6 / +2 lines)
Lines 136-147 sub delete { Link Here
136
136
137
    return try {
137
    return try {
138
        $item_link->delete;
138
        $item_link->delete;
139
        return $c->render(
139
        return $c->resource_deleted;
140
            status  => 204,
140
    } catch {
141
            openapi => q{}
142
        );
143
    }
144
    catch {
145
        $c->unhandled_exception($_);
141
        $c->unhandled_exception($_);
146
    };
142
    };
147
}
143
}
(-)a/Koha/REST/V1/Bookings.pm (-4 / +1 lines)
Lines 140-149 sub delete { Link Here
140
140
141
    return try {
141
    return try {
142
        $booking->delete;
142
        $booking->delete;
143
        return $c->render(
143
        return $c->resource_deleted;
144
            status  => 204,
145
            openapi => q{}
146
        );
147
    } catch {
144
    } catch {
148
        $c->unhandled_exception($_);
145
        $c->unhandled_exception($_);
149
    };
146
    };
(-)a/Koha/REST/V1/Cities.pm (-4 / +1 lines)
Lines 119-128 sub delete { Link Here
119
119
120
    return try {
120
    return try {
121
        $city->delete;
121
        $city->delete;
122
        return $c->render(
122
        return $c->resource_deleted;
123
            status  => 204,
124
            openapi => q{}
125
        );
126
    }
123
    }
127
    catch {
124
    catch {
128
        $c->unhandled_exception($_);
125
        $c->unhandled_exception($_);
(-)a/Koha/REST/V1/Config/SMTP/Servers.pm (-7 / +2 lines)
Lines 164-176 sub delete { Link Here
164
164
165
    return try {
165
    return try {
166
        $smtp_server->delete;
166
        $smtp_server->delete;
167
167
        return $c->resource_deleted;
168
        return $c->render(
168
    } catch {
169
            status  => 204,
170
            openapi => q{}
171
        );
172
    }
173
    catch {
174
        $c->unhandled_exception($_);
169
        $c->unhandled_exception($_);
175
    };
170
    };
176
}
171
}
(-)a/Koha/REST/V1/ERM/Agreements.pm (-6 / +2 lines)
Lines 247-258 sub delete { Link Here
247
247
248
    return try {
248
    return try {
249
        $agreement->delete;
249
        $agreement->delete;
250
        return $c->render(
250
        return $c->resource_deleted;
251
            status  => 204,
251
    } catch {
252
            openapi => q{}
253
        );
254
    }
255
    catch {
256
        $c->unhandled_exception($_);
252
        $c->unhandled_exception($_);
257
    };
253
    };
258
}
254
}
(-)a/Koha/REST/V1/ERM/EHoldings/Packages/Local.pm (-6 / +2 lines)
Lines 220-231 sub delete { Link Here
220
220
221
    return try {
221
    return try {
222
        $package->delete;
222
        $package->delete;
223
        return $c->render(
223
        return $c->resource_deleted;
224
            status  => 204,
224
    } catch {
225
            openapi => q{}
226
        );
227
    }
228
    catch {
229
        $c->unhandled_exception($_);
225
        $c->unhandled_exception($_);
230
    };
226
    };
231
}
227
}
(-)a/Koha/REST/V1/ERM/EHoldings/Titles/Local.pm (-6 / +2 lines)
Lines 216-227 sub delete { Link Here
216
216
217
    return try {
217
    return try {
218
        $title->delete;
218
        $title->delete;
219
        return $c->render(
219
        return $c->resource_deleted;
220
            status  => 204,
220
    } catch {
221
            openapi => q{}
222
        );
223
    }
224
    catch {
225
        $c->unhandled_exception($_);
221
        $c->unhandled_exception($_);
226
    };
222
    };
227
}
223
}
(-)a/Koha/REST/V1/ERM/EUsage/CounterFiles.pm (-4 / +1 lines)
Lines 85-94 sub delete { Link Here
85
85
86
    return try {
86
    return try {
87
        $counter_file->delete;
87
        $counter_file->delete;
88
        return $c->render(
88
        return $c->resource_deleted;
89
            status  => 204,
90
            openapi => q{}
91
        );
92
    } catch {
89
    } catch {
93
        $c->unhandled_exception($_);
90
        $c->unhandled_exception($_);
94
    };
91
    };
(-)a/Koha/REST/V1/ERM/EUsage/DefaultUsageReports.pm (-4 / +1 lines)
Lines 119-128 sub delete { Link Here
119
119
120
    return try {
120
    return try {
121
        $default_report->delete;
121
        $default_report->delete;
122
        return $c->render(
122
        return $c->resource_deleted;
123
            status  => 204,
124
            openapi => q{}
125
        );
126
    } catch {
123
    } catch {
127
        $c->unhandled_exception($_);
124
        $c->unhandled_exception($_);
128
    };
125
    };
(-)a/Koha/REST/V1/ERM/EUsage/UsageDataProviders.pm (-4 / +1 lines)
Lines 263-272 sub delete { Link Here
263
263
264
    return try {
264
    return try {
265
        $usage_data_provider->delete;
265
        $usage_data_provider->delete;
266
        return $c->render(
266
        return $c->resource_deleted;
267
            status  => 204,
268
            openapi => q{}
269
        );
270
    } catch {
267
    } catch {
271
        $c->unhandled_exception($_);
268
        $c->unhandled_exception($_);
272
    };
269
    };
(-)a/Koha/REST/V1/ERM/Licenses.pm (-4 / +1 lines)
Lines 228-237 sub delete { Link Here
228
228
229
    return try {
229
    return try {
230
        $license->delete;
230
        $license->delete;
231
        return $c->render(
231
        return $c->resource_deleted;
232
            status  => 204,
233
            openapi => q{}
234
        );
235
    }
232
    }
236
    catch {
233
    catch {
237
        $c->unhandled_exception($_);
234
        $c->unhandled_exception($_);
(-)a/Koha/REST/V1/Holds.pm (-5 / +2 lines)
Lines 322-331 sub delete { Link Here
322
322
323
        $hold->cancel;
323
        $hold->cancel;
324
324
325
        return $c->render(
325
        return $c->resource_deleted;
326
            status  => 204,
327
            openapi => q{}
328
        );
329
    }
326
    }
330
    catch {
327
    catch {
331
        $c->unhandled_exception($_);
328
        $c->unhandled_exception($_);
Lines 389-395 sub resume { Link Here
389
386
390
    return try {
387
    return try {
391
        $hold->resume;
388
        $hold->resume;
392
        return $c->render( status => 204, openapi => {} );
389
        return $c->resource_deleted;
393
    }
390
    }
394
    catch {
391
    catch {
395
        $c->unhandled_exception($_);
392
        $c->unhandled_exception($_);
(-)a/Koha/REST/V1/IllbatchStatuses.pm (-1 / +1 lines)
Lines 148-154 sub delete { Link Here
148
148
149
    return try {
149
    return try {
150
        $status->delete_and_log;
150
        $status->delete_and_log;
151
        return $c->render( status => 204, openapi => '' );
151
        return $c->resource_deleted;
152
    } catch {
152
    } catch {
153
        $c->unhandled_exception($_);
153
        $c->unhandled_exception($_);
154
    };
154
    };
(-)a/Koha/REST/V1/Illbatches.pm (-1 / +1 lines)
Lines 165-171 sub delete { Link Here
165
165
166
    return try {
166
    return try {
167
        $batch->delete_and_log;
167
        $batch->delete_and_log;
168
        return $c->render( status => 204, openapi => '' );
168
        return $c->resource_deleted;
169
    } catch {
169
    } catch {
170
        $c->unhandled_exception($_);
170
        $c->unhandled_exception($_);
171
    };
171
    };
(-)a/Koha/REST/V1/ImportBatchProfiles.pm (-7 / +2 lines)
Lines 121-133 sub delete { Link Here
121
121
122
    return try {
122
    return try {
123
        $profile->delete;
123
        $profile->delete;
124
124
        return $c->resource_deleted;
125
        return $c->render(
125
    } catch {
126
            status => 204,
127
            openapi => q{}
128
        );
129
    }
130
    catch {
131
        $c->unhandled_exception($_);
126
        $c->unhandled_exception($_);
132
    };
127
    };
133
}
128
}
(-)a/Koha/REST/V1/Items.pm (-9 / +2 lines)
Lines 158-168 sub delete { Link Here
158
        }
158
        }
159
159
160
        $item->safe_delete;
160
        $item->safe_delete;
161
161
        return $c->resource_deleted;
162
        return $c->render(
163
            status  => 204,
164
            openapi => q{}
165
        );
166
    }
162
    }
167
    catch {
163
    catch {
168
        $c->unhandled_exception($_);
164
        $c->unhandled_exception($_);
Lines 421-430 sub remove_from_bundle { Link Here
421
417
422
    return try {
418
    return try {
423
        $bundle_item->remove_from_bundle;
419
        $bundle_item->remove_from_bundle;
424
        return $c->render(
420
        return $c->resource_deleted;
425
            status  => 204,
426
            openapi => q{}
427
        );
428
    } catch {
421
    } catch {
429
        if ( ref($_) eq 'Koha::Exceptions::Item::Bundle::BundleIsCheckedOut' ) {
422
        if ( ref($_) eq 'Koha::Exceptions::Item::Bundle::BundleIsCheckedOut' ) {
430
            return $c->render(
423
            return $c->render(
(-)a/Koha/REST/V1/Libraries.pm (-1 / +1 lines)
Lines 152-158 sub delete { Link Here
152
152
153
    return try {
153
    return try {
154
        $library->delete;
154
        $library->delete;
155
        return $c->render( status => 204, openapi => '');
155
        return $c->resource_deleted;
156
    }
156
    }
157
    catch {
157
    catch {
158
        $c->unhandled_exception($_);
158
        $c->unhandled_exception($_);
(-)a/Koha/REST/V1/Patrons.pm (-4 / +1 lines)
Lines 414-423 sub delete { Link Here
414
                $patron->move_to_deleted;
414
                $patron->move_to_deleted;
415
                $patron->delete;
415
                $patron->delete;
416
416
417
                return $c->render(
417
                return $c->resource_deleted;
418
                    status  => 204,
419
                    openapi => q{}
420
                );
421
            }
418
            }
422
        );
419
        );
423
    } catch {
420
    } catch {
(-)a/Koha/REST/V1/Patrons/Attributes.pm (-4 / +1 lines)
Lines 285-294 sub delete { Link Here
285
            unless $attribute;
285
            unless $attribute;
286
286
287
        $attribute->delete;
287
        $attribute->delete;
288
        return $c->render(
288
        return $c->resource_deleted;
289
            status  => 204,
290
            openapi => q{}
291
        );
292
    }
289
    }
293
    catch {
290
    catch {
294
        $c->unhandled_exception($_);
291
        $c->unhandled_exception($_);
(-)a/Koha/REST/V1/Patrons/Holds.pm (-4 / +1 lines)
Lines 78-87 sub delete_public { Link Here
78
78
79
        if ( $hold->is_cancelable_from_opac ) {
79
        if ( $hold->is_cancelable_from_opac ) {
80
            $hold->cancel;
80
            $hold->cancel;
81
            return $c->render(
81
            return $c->resource_deleted;
82
                status  => 204,
83
                openapi => q{},
84
            );
85
        } elsif ( $hold->is_waiting and $hold->cancellation_requestable_from_opac ) {
82
        } elsif ( $hold->is_waiting and $hold->cancellation_requestable_from_opac ) {
86
            $hold->add_cancellation_request;
83
            $hold->add_cancellation_request;
87
            return $c->render(
84
            return $c->render(
(-)a/Koha/REST/V1/Preservation/Processings.pm (-4 / +1 lines)
Lines 208-217 sub delete { Link Here
208
    }
208
    }
209
    return try {
209
    return try {
210
        $processing->delete;
210
        $processing->delete;
211
        return $c->render(
211
        return $c->resource_deleted;
212
            status  => 204,
213
            openapi => q{}
214
        );
215
    } catch {
212
    } catch {
216
        $c->unhandled_exception($_);
213
        $c->unhandled_exception($_);
217
    };
214
    };
(-)a/Koha/REST/V1/Preservation/Trains.pm (-9 / +2 lines)
Lines 200-209 sub delete { Link Here
200
200
201
    return try {
201
    return try {
202
        $train->delete;
202
        $train->delete;
203
        return $c->render(
203
        return $c->resource_deleted;
204
            status  => 204,
205
            openapi => q{}
206
        );
207
    } catch {
204
    } catch {
208
        $c->unhandled_exception($_);
205
        $c->unhandled_exception($_);
209
    };
206
    };
Lines 509-519 sub remove_item { Link Here
509
506
510
    return try {
507
    return try {
511
        $train_item->delete;
508
        $train_item->delete;
512
509
        return $c->resource_deleted;
513
        return $c->render(
514
            status  => 204,
515
            openapi => q{}
516
        );
517
    } catch {
510
    } catch {
518
        $c->unhandled_exception($_);
511
        $c->unhandled_exception($_);
519
    };
512
    };
(-)a/Koha/REST/V1/Preservation/WaitingList.pm (-4 / +1 lines)
Lines 145-154 sub remove_item { Link Here
145
145
146
        $item->notforloan(0)->store;
146
        $item->notforloan(0)->store;
147
147
148
        return $c->render(
148
        return $c->resource_deleted;
149
            status  => 204,
150
            openapi => q{}
151
        );
152
    } catch {
149
    } catch {
153
        $c->unhandled_exception($_);
150
        $c->unhandled_exception($_);
154
    };
151
    };
(-)a/Koha/REST/V1/Quotes.pm (-4 / +1 lines)
Lines 121-130 sub delete { Link Here
121
121
122
    return try {
122
    return try {
123
        $quote->delete;
123
        $quote->delete;
124
        return $c->render(
124
        return $c->resource_deleted;
125
            status  => 204,
126
            openapi => q{}
127
        );
128
    }
125
    }
129
    catch {
126
    catch {
130
        $c->unhandled_exception($_);
127
        $c->unhandled_exception($_);
(-)a/Koha/REST/V1/ReturnClaims.pm (-6 / +2 lines)
Lines 192-203 sub delete_claim { Link Here
192
192
193
        $claim->delete();
193
        $claim->delete();
194
194
195
        return $c->render(
195
        return $c->resource_deleted;
196
            status  => 204,
196
    } catch {
197
            openapi => {}
198
        );
199
    }
200
    catch {
201
        $c->unhandled_exception($_);
197
        $c->unhandled_exception($_);
202
    };
198
    };
203
}
199
}
(-)a/Koha/REST/V1/SearchFilter.pm (-3 / +2 lines)
Lines 135-143 sub delete { Link Here
135
135
136
    return try {
136
    return try {
137
        $filter->delete;
137
        $filter->delete;
138
        return $c->render( status => 204, openapi => q{} );
138
        return $c->resource_deleted;
139
    }
139
    } catch {
140
    catch {
141
        $c->unhandled_exception($_);
140
        $c->unhandled_exception($_);
142
    };
141
    };
143
}
142
}
(-)a/Koha/REST/V1/Suggestions.pm (-6 / +2 lines)
Lines 201-212 sub delete { Link Here
201
201
202
    return try {
202
    return try {
203
        $suggestion->delete;
203
        $suggestion->delete;
204
        return $c->render(
204
        return $c->resource_deleted;
205
            status  => 204,
205
    } catch {
206
            openapi => q{}
207
        );
208
    }
209
    catch {
210
        $c->unhandled_exception($_);
206
        $c->unhandled_exception($_);
211
    };
207
    };
212
}
208
}
(-)a/Koha/REST/V1/Tickets.pm (-6 / +2 lines)
Lines 132-143 sub delete { Link Here
132
132
133
    return try {
133
    return try {
134
        $ticket->delete;
134
        $ticket->delete;
135
        return $c->render(
135
        return $c->resource_deleted;
136
            status  => 204,
136
    } catch {
137
            openapi => q{}
138
        );
139
    }
140
    catch {
141
        $c->unhandled_exception($_);
137
        $c->unhandled_exception($_);
142
    };
138
    };
143
}
139
}
(-)a/Koha/REST/V1/TransferLimits.pm (-2 / +2 lines)
Lines 109-115 sub delete { Link Here
109
109
110
    return try {
110
    return try {
111
        $transfer_limit->delete;
111
        $transfer_limit->delete;
112
        return $c->render( status => 204, openapi => '');
112
        return $c->resource_deleted;
113
    }
113
    }
114
    catch {
114
    catch {
115
        $c->unhandled_exception($_);
115
        $c->unhandled_exception($_);
Lines 217-223 sub batch_delete { Link Here
217
217
218
        Koha::Item::Transfer::Limits->search($search_params)->delete;
218
        Koha::Item::Transfer::Limits->search($search_params)->delete;
219
219
220
        return $c->render( status => 204, openapi => q{} );
220
        return $c->resource_deleted;
221
    }
221
    }
222
    catch {
222
    catch {
223
        $c->unhandled_exception($_);
223
        $c->unhandled_exception($_);
(-)a/Koha/REST/V1/TwoFactorAuth.pm (-1 / +1 lines)
Lines 173-179 sub verification { Link Here
173
            );
173
            );
174
        }
174
        }
175
175
176
        return $c->render(status => 204, openapi => {});
176
        return $c->resource_deleted;
177
    }
177
    }
178
    catch {
178
    catch {
179
        $c->unhandled_exception($_);
179
        $c->unhandled_exception($_);
(-)a/t/db_dependent/api/v1/responses.t (-2 / +1 lines)
Lines 105-111 subtest 'resource_deleted() tests' => sub { Link Here
105
        'delete',
105
        'delete',
106
        sub {
106
        sub {
107
            my $c = shift->openapi->valid_input or return;
107
            my $c = shift->openapi->valid_input or return;
108
            $c->resource_deleted;
108
            return $c->resource_deleted;
109
        }
109
        }
110
    );
110
    );
111
111
112
- 

Return to bug 36495