From 590dc7103af9b2150e452588399b8be49eb194c8 Mon Sep 17 00:00:00 2001 From: Matt Blenkinsop Date: Wed, 17 Apr 2024 16:23:33 +0000 Subject: [PATCH] Bug 36627: Record who is importing sushi data This patch adds functionality for storing the patron (or cronjob) that triggers a harvest. Currently every counter log is stored as being imported by the cronjob Test plan: 1) In the usage statistics module in ERM, create a data provider (the details in the provider are not important) 2) Click into the provider and click into manual upload 3) Upload the file attached to this bug 4) Once the background job is completed click into the counter logs tab in the provider. The imported by columns should show a user with a clickable link Signed-off-by: Pedro Amorim --- Koha/ERM/EUsage/CounterFile.pm | 5 ++++- .../ERM/UsageStatisticsDataProvidersCounterLogs.vue | 11 +++++++---- 2 files changed, 11 insertions(+), 5 deletions(-) diff --git a/Koha/ERM/EUsage/CounterFile.pm b/Koha/ERM/EUsage/CounterFile.pm index 0836455251b..2d5d80f15cd 100644 --- a/Koha/ERM/EUsage/CounterFile.pm +++ b/Koha/ERM/EUsage/CounterFile.pm @@ -35,6 +35,8 @@ use Koha::ERM::EUsage::UsageDataProvider; use Koha::ERM::EUsage::SushiCounter; use Koha::Exceptions::ERM::EUsage::CounterFile; +use C4::Context; + use base qw(Koha::Object); use Koha::ERM::EUsage::CounterLogs; @@ -558,10 +560,11 @@ Adds a erm_counter_logs database entry sub _add_counter_log_entry { my ($self) = @_; + my $user = C4::Context->userenv()->{'number'}; Koha::ERM::EUsage::CounterLog->new( { #TODO: borrowernumber only required for manual uploads or "harvest now" button clicks - borrowernumber => undef, + borrowernumber => $user, counter_files_id => $self->erm_counter_files_id, importdate => $self->date_uploaded, filename => $self->filename, diff --git a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/UsageStatisticsDataProvidersCounterLogs.vue b/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/UsageStatisticsDataProvidersCounterLogs.vue index 741545d07ef..624b792cf95 100644 --- a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/UsageStatisticsDataProvidersCounterLogs.vue +++ b/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/UsageStatisticsDataProvidersCounterLogs.vue @@ -40,7 +40,9 @@ export default { building_table: false, tableOptions: { columns: this.getTableColumns(), - options: {}, + options: { + embed: "borrowernumber", + }, url: () => this.table_url(), table_settings: this.counter_log_table_settings, add_filters: true, @@ -137,9 +139,10 @@ export default { { title: __("Imported by"), render: function (data, type, row, meta) { - const importer = row.borrowernumber - ? `Borrowernumber ${row.borrowernumber}` - : __("Cronjob") + const borrower = row.borrowernumber + const importer = borrower + ? `${borrower.firstname} ${borrower.surname}` + : this.$__("Cronjob") return importer }, searchable: true, -- 2.39.3 (Apple Git-146)