Site Links
Home
Features
Documentation
Pricing & Order
Members Area
Support Options
Who's Online
6 registered (Island Piper, jacampbe, Rick, Ruben Rocha, Thelockman, 1 invisible), 32 Guests and 19 Spiders online.
Key: Admin, Global Mod, Mod
Featured Member
Registered: 09/18/08
Posts: 9
Top Posters (30 Days)
Ruben Rocha 184
Gizmo 110
Rick 101
Thelockman 83
driv 41
AllenAyres 35
ntdoc 28
Sirdude 27
Ian 24
ScriptKeeper 18
Latest Photos
My Home System
test photo gallery
Bernese Mountain Dogs
My Daimler
Dorado and shark
Page 1 of 2 1 2 >
Topic Options
Rate This Topic
#196054 - 09/02/07 09:10 AM [Fixed 7.3][7.2.2] Merged posts don't have their correct spot in the topiclist
blaaskaak Offline

****

Registered: 08/30/06
Posts: 1299
Loc: Breda, NL
It's missing an update to the topics table with their new date/time/subject etc of the new latest post in the topic.

Both with merged topics and merged posts.


Edited by Sirdude (11/09/07 12:30 AM)
_________________________

Top
#197667 - 09/19/07 04:27 AM Re: [7.2.2] Merged posts don't have their correct spot in the topiclist [Re: blaaskaak]
Myke Offline
journeyman

Registered: 07/07/06
Posts: 83
I just ran into this bug when trying to merge a post and its replies to another topic.

In threaded mode, the thread structure looks fine but in flat mode, the order is out of whack.

blaaskaak, do you have a manual fix for this?

Top
#197668 - 09/19/07 05:28 AM Re: [7.2.2] Merged posts don't have their correct spot in the topiclist [Re: Myke]
blaaskaak Offline

****

Registered: 08/30/06
Posts: 1299
Loc: Breda, NL
No...

When you merge topics, in flatmode the order that the topic is shown is in the original posting order.

It's not that topic A is appended to topic B, they are merged, as in mixed together if both topics ran simultaniously.

The bug repored is about when you merge topic A to topic B, topic B's last post date/time/subject is not updated to reflect the latest topic in that topic.
_________________________

Top
#197669 - 09/19/07 06:00 AM Re: [7.2.2] Merged posts don't have their correct spot in the topiclist [Re: blaaskaak]
Myke Offline
journeyman

Registered: 07/07/06
Posts: 83
Hmm, maybe I'm reporting a different bug then?

Here's what happened with me. I had two topics, say topic A and topic B.

Topic A had 6 replies:

A.0, A.1, A.2, A.3, A.4, A.5

Topic B had 0 replies (i.e. just the single post):

B.0

I merged Topic A to Topic B, and when viewing the end result in Flat Mode, I got the following:

A.0, B.0, A.1, A.2, A.3, A.4, A.5

What I expected to see was:

B.0, A.0, A.1, A.2, A.3, A.4, A.5

Looking in showflat.php, I notice that the main query just sorts by the Post ID, which is effectively sorting chronologically:

 Code:
ORDER BY POST_ID


But this doesn't guarantee that the main parent post of a topic gets displayed first! As shown in my example above.

So I changed it such that the main parent post will always appear first, and then the remaining posts are in chronological order:

 Code:
ORDER BY POST_IS_TOPIC DESC, POST_ID


So, I still think I have found a bug in that the parent post (where POST_IS_TOPIC=1) isn't gauranteed to display first in Flat Mode.

Top
#197670 - 09/19/07 06:22 AM Re: [7.2.2] Merged posts don't have their correct spot in the topiclist [Re: Myke]
blaaskaak Offline

****

Registered: 08/30/06
Posts: 1299
Loc: Breda, NL
Don't know if you can consider it a bug. Merging is merging, 2 seperate topics become one. The only difference in merging topic A to B or B to A is the one that might get "This topic has merged" topic.

In flatmode, the normal vieworder is chronologically, so I think it works as designed.

What could be doable is an extra option called "Append topics", where a topic is appended to the target topic.

In normal day life, this isn't so much of a bother to use. We mainly use merge when somebody opens a new topic when there is already an older one about the same subject. You just merge them, and no troubles there (well, except bug #1 in this topic).
_________________________

Top
#197671 - 09/19/07 06:43 AM Re: [7.2.2] Merged posts don't have their correct spot in the topiclist [Re: blaaskaak]
Myke Offline
journeyman

Registered: 07/07/06
Posts: 83
Consider this then:

  • You're viewing the post list in the Changelog forum
  • You click on the topic titled "Current Changelog" started by Rick
  • The next thing you see is Rick's post, as expected. But it appears half-way down the page.
  • When you scroll up you see there are other posts before Rick's.

Bug or not a bug?

Who started the topic? \:\)

p.s. assume there are no new posts in the thread, and the posts appearing before Rick were not posted by him.


Edited by Myke (09/19/07 06:44 AM)
Edit Reason: added assumption

Top
#197673 - 09/19/07 07:48 AM Re: [7.2.2] Merged posts don't have their correct spot in the topiclist [Re: Myke]
blaaskaak Offline

****

Registered: 08/30/06
Posts: 1299
Loc: Breda, NL
 Originally Posted By: Myke
Who started the topic? \:\)


My situation

Topic 1
Topic 2
Topic 3
Topic 4
Topic 5
Topic 7
Topic 8
Topic 9

Merge topic 2 into topic 8... result:

Topic 1
Topic 3
Topic 4
Topic 5
Topic 7
Topic 8
Topic 9

Topic 8 should be on the spot where topic 2 used to be.
_________________________

Top
#197674 - 09/19/07 08:05 AM Re: [7.2.2] Merged posts don't have their correct spot in the topiclist [Re: blaaskaak]
Rick Administrator Online   content

*****

Registered: 06/04/06
Posts: 7904
Loc: Aberdeen, WA
Yeah, what I think we're going to have to do is actually create an entirely new topic out of the merged topics. Right now we're just changing the topic id of the posts, and it's leading to some inconsistencies with how the merged topics are displayed.
_________________________
UBB.threads™ Developer
My Personal Website · StogieSmokers.com

Top
#197675 - 09/19/07 08:40 AM Re: [7.2.2] Merged posts don't have their correct spot in the topiclist [Re: Rick]
Ian Spence Moderator Offline


Registered: 06/04/06
Posts: 255
Rick, I actually think I might have fixed this in 7.3. We just needed to run rebuild_topic_data($topic_id) for the topic that was merged into.
_________________________
My UBB7 Forums

Top
#197689 - 09/19/07 02:14 PM Re: [7.2.2] Merged posts don't have their correct spot in the topiclist [Re: Ian Spence]
Rick Administrator Online   content

*****

Registered: 06/04/06
Posts: 7904
Loc: Aberdeen, WA
Ah cool, thanks Ian ;\)
_________________________
UBB.threads™ Developer
My Personal Website · StogieSmokers.com

Top
Page 1 of 2 1 2 >


Shout Box

Today's Birthdays
kimteague_5@msn.com, Trekkie, Trixie
Recent Topics
Inclusion of Locked/Closed thread capabilities
by QSS Tim
12/03/08 11:17 AM
Contact Page Problem
by David DelMonte
12/03/08 09:05 AM
FAQ bug,
by BreeOge
12/03/08 08:36 AM
I'm the Admin but.....
by David DelMonte
12/02/08 11:12 AM
Fatal error: Smarty::require_once
()

by ThreadsUser
12/02/08 08:17 AM
Forum Stats
4261 Members
33 Forums
30538 Topics
155004 Posts

Max Online: 978 @ 06/24/07 08:19 PM