Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sort error for banlist.inc.php #3452

Closed
tmp2000 opened this issue Nov 3, 2016 · 2 comments
Closed

Sort error for banlist.inc.php #3452

tmp2000 opened this issue Nov 3, 2016 · 2 comments
Labels

Comments

@tmp2000
Copy link

tmp2000 commented Nov 3, 2016

Under develop branch and 1.9.8.1 version(might other version also have the same error), line 113-116 in banlist.inc.php , the string "staff.php" should be changed to "banlist.php" .Otherwise ,the sort would not work which mentioned in issue #1725 .

            <th width="56%"><a <?php echo $email_sort; ?> href="staff.php?<?php echo $qstr; ?>&sort=email"><?php echo __('Email Address');?></a></th>
            <th width="10%"><a  <?php echo $status_sort; ?> href="staff.php?<?php echo $qstr; ?>&sort=status"><?php echo __('Ban Status');?></a></th>
            <th width="10%"><a <?php echo $created_sort; ?> href="staff.php?<?php echo $qstr; ?>&sort=created"><?php echo __('Date Added');?></a></th>
            <th width="20%"><a <?php echo $updated_sort; ?> href="staff.php?<?php echo $qstr; ?>&sort=updated"><?php echo __('Last Updated');?></a></th>

to

        <th width="56%"><a <?php echo $email_sort; ?> href="banlist.php?<?php echo $qstr; ?>&sort=email"><?php echo __('Email Address');?></a></th>
        <th width="10%"><a  <?php echo $status_sort; ?> href="banlist.php?<?php echo $qstr; ?>&sort=status"><?php echo __('Ban Status');?></a></th>
        <th width="10%"><a <?php echo $created_sort; ?> href="banlist.php?<?php echo $qstr; ?>&sort=created"><?php echo __('Date Added');?></a></th>
        <th width="20%"><a <?php echo $updated_sort; ?> href="banlist.php?<?php echo $qstr; ?>&sort=updated"><?php echo __('Last Updated');?></a></th>
@ntozier
Copy link
Contributor

ntozier commented Nov 3, 2016

So 1.9.8.1 is really old at this point. You should really update your version.

JediKev added a commit to JediKev/osTicket that referenced this issue Nov 3, 2016
This addresses issue osTicket#3452 where upon sorting email banlist it redirects to Agents tab.
JediKev added a commit to JediKev/osTicket that referenced this issue Nov 3, 2016
This addresses issue osTicket#3452 where upon sorting email banlist it redirects to Agents tab.
@JediKev
Copy link
Contributor

JediKev commented Nov 3, 2016

I have a Pull Request that is pending on this issue. Thank you.

@FossPrime FossPrime added the bug label Dec 8, 2016
kurtbr pushed a commit to kurtbr/osTicket that referenced this issue Dec 11, 2019
This addresses issue osTicket#3452 where upon sorting email banlist it redirects to Agents tab.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants