http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9458

--- Comment #25 from Kyle M Hall <[email protected]> ---
> I am a little bit perplexed by this line. Why did you add the || 'title'
> when $sortfield has already been set? Or, equivalently, why did you set
> $sortfield previously? I would have thought the simplest way to get the
> desired results would be:
>   my $sortfield = $query->param('sort') || $sorton || 'title';
> 
> My chief concern is, is there some sort of side-effect in the code that
> you're trying to work around with this?

Nope, that's just me being stupid ; ) 

I'll post a small followup to fix that.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to