Maybe this is my fault. But i think it other developer just make the same mistake.
XenForo/Model/Thread.php line 109
please change to:
if (!isset($fetchOptions['orderDirection']) || $fetchOptions['orderDirection'] == 'desc')
to
if (!isset($fetchOptions['orderDirection']) || strtolower($fetchOptions['orderDirection']) == 'desc')
Maybe there are more positions in different models.
I just want to start a discussion and hope to make it easier for developer to use models in their addons.
Thanks,
Daniel
XenForo/Model/Thread.php line 109
please change to:
if (!isset($fetchOptions['orderDirection']) || $fetchOptions['orderDirection'] == 'desc')
to
if (!isset($fetchOptions['orderDirection']) || strtolower($fetchOptions['orderDirection']) == 'desc')
Maybe there are more positions in different models.
I just want to start a discussion and hope to make it easier for developer to use models in their addons.
Thanks,
Daniel
Upvote
0