Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incorrect allocation of comments and posts #11890

Closed
atasingorneq opened this issue Sep 5, 2022 · 4 comments · Fixed by #11893
Closed

Incorrect allocation of comments and posts #11890

atasingorneq opened this issue Sep 5, 2022 · 4 comments · Fixed by #11893
Labels

Comments

@atasingorneq
Copy link

  • [x ] I have searched open and closed issues for duplicates

Bug Description

I think there are big problems with assigning comments to launch posts right now. Comments on different posts are combined into one thread.

Some examples:

Steps to Reproduce

Comment on one of the posts and see what happens.

Actual Result:

A mixed thread is created, the initial post and comments do not fit together.

Expected Result:

The comments are assigned to the correct posts

Screenshots

Platform Info

The wrong thread
falsche-Threadzuordnung

The right thread
richtiger-thread

Friendica Version:
Friendica 'Giant Rhubarb' 2022.09-rc - 1482

Friendica Source: git

PHP version: 8.0.23

@annando
Copy link
Collaborator

annando commented Sep 5, 2022

Please execute this query:

select `parent-uri`, count(*) as parents from `post-view` where gravity = 0 group by `parent-uri` having parents > 1;

@atasingorneq
Copy link
Author

atasingorneq commented Sep 5, 2022

The result of the query

select `parent-uri`, count(*) as parents from `post-view` where gravity = 0 group by `parent-uri` having parents > 1;
Empty set (12.713 sec)`

@xundeenergie
Copy link

This is my result. Threadings are also broken on my side

MariaDB [friendica]> select `parent-uri`, count(*) as parents from `post-view` where gravity = 0 group by `parent-uri` having parents > 1;
+-----------------------------------------------------------------------+---------+
| parent-uri                                                            | parents |
+-----------------------------------------------------------------------+---------+
| https://digitalcourage.social/users/mupan/statuses/108793717950243445 |       2 |
| https://embassy.social/users/Thalestria/statuses/108783150929196474   |       2 |
| https://fem.social/users/mehdorn/statuses/107693834556002392          |       2 |
| https://norden.social/users/Spreepunk/statuses/108927995631015683     |       2 |
| https://squeet.me/objects/962c3e10-1563-1453-4e13-722767753025        |       3 |
| https://troet.cafe/users/dieAnnie/statuses/108849093696422351         |       2 |
| https://troet.cafe/users/morsuapri/statuses/108913293043179533        |       2 |
+-----------------------------------------------------------------------+---------+
7 rows in set (9.002 sec)

@xundeenergie
Copy link

It happened after the last checkout in devel branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants