[Xoops-development] newbb minor bug

[R. M. v. D. <mv...@ca...>]

A user on xoops.org, ScoobRS, has pointed out a minor bug with newbb...

(1) After submitting a 'post', you are redirected to a page with 'jump=1'
which gives the thread in newest first order.  However the links to
additional pages in the thread (1, 2, 3 at bottom) give the thread in
oldest first order.  This is inconsistent and confusing.

(2) The notification email and forum index try to jump to the correct post
(via an anchor #forumpostXYZ) but for a multi-page thread, this does not
automatically jump to the correct page.

A possible fix is to:

- after 'post', go to oldest-first thread order and jump to the correct
post on the last page of the thread

- the page-number links should take into account the current 'order' of
the thread an be consistent with it

- the notification email and forum index links should also jump to the
correct page

The only worry is that, in some cases, we may need an extra query to get
the number of posts in the thread.  During a 'post' operation I don't
think this is critical, but during viewing of the thread it should be
avoided.  Perhaps there is some way to get this info by tapping into one
of the existing queries.

Any other thoughts?

Mike