$pagingbar .= get_string('page').': ';
$sistrings = array();
- $sistrings[] = $sifirst != 'all' ? "sifirst={$sifirst}" : null;
- $sistrings[] = $silast != 'all' ? "silast={$silast}" : null;
- $sistring = !empty($sistrings) ? implode('&', $sistrings) : '';
+ if ($sifirst != 'all') {
+ $sistrings[] = "sifirst={$sifirst}";
+ }
+ if ($silast != 'all') {
+ $sistrings[] = "silast={$silast}";
+ }
+ $sistring = !empty($sistrings) ? '&'.implode('&', $sistrings) : '';
// Display previous link
if ($start > 0) {
$pagingbar .= get_string('page').': ';
$sistrings = array();
- $sistrings[] = $sifirst != 'all' ? "sifirst={$sifirst}" : null;
- $sistrings[] = $silast != 'all' ? "silast={$silast}" : null;
- $sistring = !empty($sistrings) ? implode('&', $sistrings) : '';
+ if ($sifirst != 'all') {
+ $sistrings[] = "sifirst={$sifirst}";
+ }
+ if ($silast != 'all') {
+ $sistrings[] = "silast={$silast}";
+ }
+ $sistring = !empty($sistrings) ? '&'.implode('&', $sistrings) : '';
// Display previous link
if ($start > 0) {