Skip to content

Commit

Permalink
Bug 14330: Remove unused email_sender from sendbasket/sendshelf
Browse files Browse the repository at this point in the history
The sendbasket/sendshelf scripts and templates do not use email_sender
as a cgi parameter or as a template var. Probably a leftover from previous
changes.
Let's make Koha cleaner :)

Test plan:
[1] Send your cart from opac or staff.
[2] Send a shelf from opac or staff.
[3] Git grep email_sender. No results.

Followed test plan. Works as expected.
Signed-off-by: Marc Véron <[email protected]>

Signed-off-by: Katrin Fischer <[email protected]>
Signed-off-by: Tomas Cohen Arazi <[email protected]>
  • Loading branch information
mderooy authored and tomascohen committed Jun 8, 2015
1 parent 34fe5c2 commit 0114465
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 6 deletions.
2 changes: 0 additions & 2 deletions basket/sendbasket.pl
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,6 @@

my $bib_list = $query->param('bib_list');
my $email_add = $query->param('email_add');
my $email_sender = $query->param('email_sender');

my $dbh = C4::Context->dbh;

Expand Down Expand Up @@ -101,7 +100,6 @@
my $resultsarray = \@results;
$template2->param(
BIBLIO_RESULTS => $resultsarray,
email_sender => $email_sender,
comment => $comment
);

Expand Down
2 changes: 0 additions & 2 deletions opac/opac-sendbasket.pl
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,6 @@

my $bib_list = $query->param('bib_list');
my $email_add = $query->param('email_add');
my $email_sender = $query->param('email_sender');

my $dbh = C4::Context->dbh;

Expand Down Expand Up @@ -115,7 +114,6 @@

$template2->param(
BIBLIO_RESULTS => $resultsarray,
email_sender => $email_sender,
comment => $comment,
firstname => $user->{firstname},
surname => $user->{surname},
Expand Down
1 change: 0 additions & 1 deletion opac/opac-sendshelf.pl
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,6 @@

$template2->param(
BIBLIO_RESULTS => \@results,
email_sender => $mail{'from'},
comment => $comment,
shelfname => $shelf[1],
firstname => $user->{firstname},
Expand Down
1 change: 0 additions & 1 deletion virtualshelves/sendshelf.pl
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,6 @@

$template2->param(
BIBLIO_RESULTS => \@results,
email_sender => $mail{'from'},
comment => $comment,
shelfname => $shelf[1],
);
Expand Down

0 comments on commit 0114465

Please sign in to comment.