Bug in pagination php?

-- url of the page with the problem -- : mywebsite.com/mypage
-- HikaShop version -- : 2.3
-- Joomla version -- : 3.2.3
-- PHP version -- : x.x.x
-- Browser(s) name and version -- : XXXXX x.x.x
-- Error-message(debug-mod must be tuned on) -- : Error_message

I just wanted to override the pagination.php which is supported since 2.1. I wondered why the pagination_list_footer didn't get called. I found this code in pagination.php which should prevent the override if you are in admin interface, if I understand it right.
Now I'm on frontend and because of the ! the pagination_list_footer would never be called. At my current understanding the code should look like this.

Am I wrong?

No, it's not a bug. We purposely not allow the override to avoid issues with the pagination of the template which can sometimes create problems with HikaShop.
So you could remove these two lines to allow the override on the frontend, but it might not work properly with some templates.

I'm writing my own template based on blank template. What kind of issues you mean?

Then also this lines (32 pagination.php) need to be removed
but then I end at a problem I wrote in another topic. www.hikashop.com/support/forum/developme...gination-overri.html

Hi,

If you want to override the items in the backend too, yes.

Do not remove the line but use instead Regards,

In this special case the frontend would not be overriden with app->isAdmin()

What we're saying basically is that templates made for Joomla and which override the joomla pagination for example don't expect to have to handle the $this->hikaSuffix and $this->form variables. So we can't allow the override on the frontend by default because it would break with most templates.
What I would recommend to do is to create a file templates/YOUR_TEMPLATE/html/com_hikashop/administrator/helpers/pagination.override.php
which would first include_once our helper.pagination file and then implement a class hikashopPaginationHelperOverride extending our hikashopPaginationHelper class so that you can redefine what you want easily in the pagination of HikaShop in your template.
That way, you have the total control on what to do and what to change in the code of the HikaShop pagination.

Thx for the hint. I've done the most stuff now by str_replace.

only the this->admin I mentioned in the first post I changed now in the hika pagination