Bug Transaction Log Search

Status
Not open for further replies.

ichpen

Customer
Use case:
Search for all transactions today for a specified user.

Transaction Log->Search:
Date range today->today, source user (user A), target user (user A) - or any combination of those.

All other parameters left blank or ANY.

Expected:
All relevant transactions for user A

What I received:
Only Login Event shown.
 
This is actually a bug in XF2. If you look in \XF\Searcher\AbstractSearcher you'll see this:

Code:
        if ($format == 'date')
        {
            if (is_array($value))
            {
                if (isset($value['start']))
                {
                    $value['start'] = $this->convertDateToInteger($value['start']);
                }
                if (isset($value['end']))
                {
                    $value['end'] = $this->convertDateToInteger($value['end']);
                }
            }
            else
            {
                $value = $this->convertDateToInteger($value);
            }
        }
The convertDateToInteger function actually does support an "end" parameter: protected function convertDateToInteger($date, $dayEnd = false) yet it's not used.

Therefore, the "today-today" searcher will only find transactions that happened at exactly 00:00:00, which is only the daily events.

I've reported this bug: https://xenforo.com/community/threa...e-not-converted-properly.156778/#post-1299912
 
Hello @ichpen,

We hope your ticket regarding DragonByte Credits has been addressed to your satisfaction. This ticket has now been closed.

If your ticket has not been resolved, you can reply to this thread at any point in the next 7 days in order to reopen the ticket, afterwards this thread will be closed.

Please do not reply to this thread if your ticket has been resolved.

Thank you.


- DragonByte Technologies, Ltd.
 
Status
Not open for further replies.

DragonByte Credits

XenForo 1.5.3+ XenForo 2.0.x XenForo 2.1.x XenForo 2.2.x
Seller
DragonByte Technologies
Release date
Last update
Total downloads
4,257
Customer rating
5.00 star(s) 4 ratings
Back
Top