[XenConcept] Next Previous Thread

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

No permission to buy ($15.00)

rdn

Well-known member
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?
 

rdn

Well-known member
Thanks a lot.

Maybe use color palette variable instead of using fixed color.
Right now, styling isn't readable on Dark Style.
Thanks!
 

rdn

Well-known member
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!
 

rdn

Well-known member
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) {
  }
}
 

XenConcept

Well-known member
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?
 

penturners

Active member
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.
 

XenConcept

Well-known member
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.
 

Sperber

Well-known member
@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?
 

XenConcept

Well-known member
@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?
 

Sperber

Well-known member
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