[XenConcept] Next Previous Thread

[XenConcept] Next Previous Thread [Paid] 2.0.2 Patch Level 3

No permission to buy ($15.00)
Bug?
Thread title having this character "&" are display on the button as: &

Also, the button isn't clickable, just the text title.
Is this intended?
 
Thanks a lot.

Maybe use color palette variable instead of using fixed color.
Right now, styling isn't readable on Dark Style.
Thanks!
 
And it has a shadow on Hover also.

Maybe just use the built-in Class for the buttons:
pageNavSimple-el pageNavSimple-el--prev pageNavSimple-el--next

So it will work in all style? :)
Thanks!
 
An error related to this addon:
Code:
LogicException: Attempted to set 'alerts_unread' while a save was pending without forceSet src/XF/Mvc/Entity/Entity.php:575
Generated by: User Oct 26, 2019 at 1:59 PM

Stack trace
#0 src/XF/Mvc/Entity/Entity.php(548): XF\Mvc\Entity\Entity->set('alerts_unread', -1)
#1 src/XF/Repository/UserAlert.php(319): XF\Mvc\Entity\Entity->__set('alerts_unread', -1)
#2 src/XF/Db/AbstractAdapter.php(455): XF\Repository\UserAlert->XF\Repository\{closure}(Object(XF\Db\Mysqli\Adapter))
#3 src/XF/Db/AbstractAdapter.php(464): XF\Db\AbstractAdapter->executeTransaction(Object(Closure), 0)
#4 src/XF/Repository/UserAlert.php(321): XF\Db\AbstractAdapter->executeTransaction(Object(Closure), 1)
#5 src/XF/Pub/Controller/Thread.php(57): XF\Repository\UserAlert->markUserAlertsReadForContent('post', Array)
#6 src/addons/LiamW/MentionMiniAvatar/XF/Pub/Controller/Thread.php(14): XF\Pub\Controller\Thread->actionIndex(Object(XF\Mvc\ParameterBag))
#7 src/addons/XFMG/XF/Pub/Controller/Thread.php(11): LiamW\MentionMiniAvatar\XF\Pub\Controller\Thread->actionIndex(Object(XF\Mvc\ParameterBag))
#8 src/addons/XenConcept/NextPreviousThread/XF/Pub/Controller/Thread.php(20): XFMG\XF\Pub\Controller\Thread->actionIndex(Object(XF\Mvc\ParameterBag))
#9 src/addons/XenConcept/HideBBCode/XF/Pub/Controller/Thread.php(21): XenConcept\NextPreviousThread\XF\Pub\Controller\Thread->actionIndex(Object(XF\Mvc\ParameterBag))
#10 src/addons/SV/ElasticSearchEssentials/XF/Pub/Controller/Thread.php(23): XenConcept\HideBBCode\XF\Pub\Controller\Thread->actionIndex(Object(XF\Mvc\ParameterBag))
#11 src/XF/Mvc/Dispatcher.php(321): SV\ElasticSearchEssentials\XF\Pub\Controller\Thread->actionIndex(Object(XF\Mvc\ParameterBag))
#12 src/XF/Mvc/Dispatcher.php(244): XF\Mvc\Dispatcher->dispatchClass('XF:Thread', 'Index', Object(XF\Mvc\RouteMatch), Object(SV\ElasticSearchEssentials\XF\Pub\Controller\Thread), NULL)
#13 src/XF/Mvc/Dispatcher.php(100): XF\Mvc\Dispatcher->dispatchFromMatch(Object(XF\Mvc\RouteMatch), Object(SV\ElasticSearchEssentials\XF\Pub\Controller\Thread), NULL)
#14 src/XF/Mvc/Dispatcher.php(50): XF\Mvc\Dispatcher->dispatchLoop(Object(XF\Mvc\RouteMatch))
#15 src/XF/App.php(2178): XF\Mvc\Dispatcher->run()
#16 src/XF.php(390): XF\App->run()
#17 index.php(20): XF::runApp('XF\\Pub\\App')
#18 {main}

Request state
array(4) {
  ["url"] => string(22) "/threads/784263/page-6"
  ["referrer"] => string(42) "https://domain.com/threads/784263/page-6"
  ["_GET"] => array(1) {
    ["/threads/784263/page-6"] => string(0) ""
  }
  ["_POST"] => array(0) {
  }
}
 
An error related to this addon:
Code:
LogicException: Attempted to set 'alerts_unread' while a save was pending without forceSet src/XF/Mvc/Entity/Entity.php:575
Generated by: User Oct 26, 2019 at 1:59 PM

Stack trace
#0 src/XF/Mvc/Entity/Entity.php(548): XF\Mvc\Entity\Entity->set('alerts_unread', -1)
#1 src/XF/Repository/UserAlert.php(319): XF\Mvc\Entity\Entity->__set('alerts_unread', -1)
#2 src/XF/Db/AbstractAdapter.php(455): XF\Repository\UserAlert->XF\Repository\{closure}(Object(XF\Db\Mysqli\Adapter))
#3 src/XF/Db/AbstractAdapter.php(464): XF\Db\AbstractAdapter->executeTransaction(Object(Closure), 0)
#4 src/XF/Repository/UserAlert.php(321): XF\Db\AbstractAdapter->executeTransaction(Object(Closure), 1)
#5 src/XF/Pub/Controller/Thread.php(57): XF\Repository\UserAlert->markUserAlertsReadForContent('post', Array)
#6 src/addons/LiamW/MentionMiniAvatar/XF/Pub/Controller/Thread.php(14): XF\Pub\Controller\Thread->actionIndex(Object(XF\Mvc\ParameterBag))
#7 src/addons/XFMG/XF/Pub/Controller/Thread.php(11): LiamW\MentionMiniAvatar\XF\Pub\Controller\Thread->actionIndex(Object(XF\Mvc\ParameterBag))
#8 src/addons/XenConcept/NextPreviousThread/XF/Pub/Controller/Thread.php(20): XFMG\XF\Pub\Controller\Thread->actionIndex(Object(XF\Mvc\ParameterBag))
#9 src/addons/XenConcept/HideBBCode/XF/Pub/Controller/Thread.php(21): XenConcept\NextPreviousThread\XF\Pub\Controller\Thread->actionIndex(Object(XF\Mvc\ParameterBag))
#10 src/addons/SV/ElasticSearchEssentials/XF/Pub/Controller/Thread.php(23): XenConcept\HideBBCode\XF\Pub\Controller\Thread->actionIndex(Object(XF\Mvc\ParameterBag))
#11 src/XF/Mvc/Dispatcher.php(321): SV\ElasticSearchEssentials\XF\Pub\Controller\Thread->actionIndex(Object(XF\Mvc\ParameterBag))
#12 src/XF/Mvc/Dispatcher.php(244): XF\Mvc\Dispatcher->dispatchClass('XF:Thread', 'Index', Object(XF\Mvc\RouteMatch), Object(SV\ElasticSearchEssentials\XF\Pub\Controller\Thread), NULL)
#13 src/XF/Mvc/Dispatcher.php(100): XF\Mvc\Dispatcher->dispatchFromMatch(Object(XF\Mvc\RouteMatch), Object(SV\ElasticSearchEssentials\XF\Pub\Controller\Thread), NULL)
#14 src/XF/Mvc/Dispatcher.php(50): XF\Mvc\Dispatcher->dispatchLoop(Object(XF\Mvc\RouteMatch))
#15 src/XF/App.php(2178): XF\Mvc\Dispatcher->run()
#16 src/XF.php(390): XF\App->run()
#17 index.php(20): XF::runApp('XF\\Pub\\App')
#18 {main}

Request state
array(4) {
  ["url"] => string(22) "/threads/784263/page-6"
  ["referrer"] => string(42) "https://domain.com/threads/784263/page-6"
  ["_GET"] => array(1) {
    ["/threads/784263/page-6"] => string(0) ""
  }
  ["_POST"] => array(0) {
  }
}

The problem seems to come from another add-on. Have you tried to disable them one by one?
 
Seems to be a well-written addon.

The next and previous threads come only from the current forum. An option for that to be site-wide would be nice.

When the option is used to replace prev/next with the thread titles, it would be nice to be able to prepend "Previous:" and "Next:" to the title. New users don't necessarily know what those titles in that position mean, and that would clearly explain what they are.

An additional top position could be the block-outer div (where the "Jump to new" and "Watch" buttons are) to consume less vertical space.
 
Seems to be a well-written addon.

The next and previous threads come only from the current forum. An option for that to be site-wide would be nice.

When the option is used to replace prev/next with the thread titles, it would be nice to be able to prepend "Previous:" and "Next:" to the title. New users don't necessarily know what those titles in that position mean, and that would clearly explain what they are.

An additional top position could be the block-outer div (where the "Jump to new" and "Watch" buttons are) to consume less vertical space.
Hello,
I take note of your comments. I will consider your request.
 
@XenConcept ,
there seems to be a bug, when posts have a prefix. If they have one, the next/previous link for this post is gone. Could you check that please?

Hello,

I can't reproduce your bug. Do you have the procedure to reproduce the bug? The add-on does not interact with prefixes. Do you have access to your site to see the error?
 
It´s a bit weired, Remi. The issue seems to be gone, as soon as the threads get rebuild. I was experiencing the missing link while importing the very first RSS feeds with prefixes. With the next feed import, the problem persisted and after an hour it was gone. So my best guess is, that one of the hourly clean-up and maintenance crons resolved it. For now I would suggest, disregard my above post.
 
Top Bottom