Bug 35477: Show a warning when non-existent items are added to the waiting list
Only messages "X new items added" or "No items added" were displayed.
With this patch we will display:
A warning with "No items added"
A warning with "X new items added. Y items not found." when some items
have not been added
A messages "X new items added" when everything went well
Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
(cherry picked from commit ed37c7e68d
)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
This commit is contained in:
parent
fec97b4ae8
commit
d7e1136020
1 changed files with 27 additions and 9 deletions
|
@ -135,13 +135,19 @@ export default {
|
|||
const PreservationStore = inject("PreservationStore")
|
||||
const { config } = PreservationStore
|
||||
|
||||
const { setMessage, setConfirmationDialog, loading, loaded } =
|
||||
inject("mainStore")
|
||||
const {
|
||||
setMessage,
|
||||
setWarning,
|
||||
setConfirmationDialog,
|
||||
loading,
|
||||
loaded,
|
||||
} = inject("mainStore")
|
||||
|
||||
return {
|
||||
table,
|
||||
config,
|
||||
setMessage,
|
||||
setWarning,
|
||||
setConfirmationDialog,
|
||||
loading,
|
||||
loaded,
|
||||
|
@ -213,12 +219,24 @@ export default {
|
|||
client.waiting_list_items.createAll(items).then(
|
||||
result => {
|
||||
if (result.length) {
|
||||
this.setMessage(
|
||||
this.$__("%s new items added.").format(
|
||||
result.length
|
||||
),
|
||||
true
|
||||
)
|
||||
if (result.length != items.length) {
|
||||
this.setWarning(
|
||||
this.$__(
|
||||
"%s new items added. %s items not found."
|
||||
).format(
|
||||
result.length,
|
||||
items.length - result.length
|
||||
),
|
||||
true
|
||||
)
|
||||
} else {
|
||||
this.setMessage(
|
||||
this.$__("%s new items added.").format(
|
||||
result.length
|
||||
),
|
||||
true
|
||||
)
|
||||
}
|
||||
this.last_items = result
|
||||
if (this.$refs.table) {
|
||||
this.$refs.table.redraw(
|
||||
|
@ -228,7 +246,7 @@ export default {
|
|||
this.getCountWaitingListItems()
|
||||
}
|
||||
} else {
|
||||
this.setMessage(this.$__("No items added"))
|
||||
this.setWarning(this.$__("No items added"))
|
||||
}
|
||||
},
|
||||
error => {}
|
||||
|
|
Loading…
Reference in a new issue