Skip to content

Commit

Permalink
fix: $request_ref->{count} not used (#6914)
Browse files Browse the repository at this point in the history
$request_ref->{count} shows up 4 times in the code. Though it never
appears to be used as three of the times that it appears it is only to
define it. The remaining instance that it appears such that it is
referenced, does not seem to benefit from the previous three instances
where the value is defined. As such this appears to be unused. This
commit removes all four instances.
  • Loading branch information
dipietroR committed Jun 20, 2022
1 parent 2542954 commit 3c12065
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
2 changes: 1 addition & 1 deletion cgi/search.pl
Original file line number Diff line number Diff line change
Expand Up @@ -768,7 +768,7 @@
if (param('search_terms')) {
open (my $OUT, ">>:encoding(UTF-8)", "$data_root/logs/search_log");
print $OUT remote_addr() . "\t" . time() . "\t" . decode utf8=>param('search_terms')
. "\tpage: $page\tcount:" . $request_ref->{count} . "\n";
. "\tpage: $page\n";
close ($OUT);
}
}
Expand Down
3 changes: 0 additions & 3 deletions lib/ProductOpener/Display.pm
Original file line number Diff line number Diff line change
Expand Up @@ -6474,7 +6474,6 @@ sub search_and_graph_products($$$) {

my @products = $cursor->all;
my $count = @products;
$request_ref->{count} = $count;

my $html = '';

Expand Down Expand Up @@ -6608,7 +6607,6 @@ sub search_and_map_products($$$) {

my @products = $cursor->all;
my $count = @products;
$request_ref->{count} = $count;

my $html = '';

Expand Down Expand Up @@ -11093,7 +11091,6 @@ sub search_and_analyze_recipes($$) {

my @products = $cursor->all;
my $count = @products;
$request_ref->{count} = $count;

my $html = '';

Expand Down

0 comments on commit 3c12065

Please sign in to comment.