Skip to content
This repository has been archived by the owner on Jan 31, 2024. It is now read-only.

Including files tab contains posts that do not include files #118

Closed
codingneko opened this issue Oct 27, 2023 · 2 comments
Closed

Including files tab contains posts that do not include files #118

codingneko opened this issue Oct 27, 2023 · 2 comments

Comments

@codingneko
Copy link

💡 Summary

The "Including files" tab in user notes contains posts that do NOT have a file because they reply to posts that do contain a file.

🥰 Expected Behavior

Only replies / posts that contain files should be shown, the post being replied to should not be a factor.

🤬 Actual Behavior

The Including Files tab is filled with replies to posts that contain a file, but that have no files, making the posts that actully have a file much harder to find.

📝 Steps to Reproduce

Access a profile from a user that has posted images or videos, and that has replied to posts with images or videos (most profiles fit this criteria), click on the "Notes" tab, and click on the "Including files" tab. Look at the posts that show up. Most of them do not have any files, they just reply to posts that do.

📌 Environment

2023.10.3.beta1

💻 Frontend

Currently on this, but affects all devices probably.

  • Model and OS of the device(s):
    Windows 11 Developer Preview
  • Browser:
    Chrome 118.0.5993.118
  • Server URL:
    https://misskey.codingneko.com
  • Sharkey:
    2023.10.3.beta1

🛰 Backend (for server admin)

Self-hosted on bare-metal server running Ubuntu 22.04.3 LTS x86_64

  • Installation Method or Hosting Service:
  • Dockerised with modified docker-compose.yml script (build .) and de-rounded (see https://misskey.codingneko.com/Sharkey at develop 4e07839)
  • OS and Architecture: Ubuntu 22.04.3 LTS x86_64
@codingneko
Copy link
Author

codingneko commented Oct 27, 2023

Acutally I'm not sure this is related to the original post having an image anymore... these do not have a file at all... I think the filter might just straight up not be working at all
image
screenshotted profile for reference: https://misskey.codingneko.com/@vaeleery@neovibe.app

@dakkar
Copy link
Contributor

dakkar commented Oct 31, 2023

I think I fixed this, see #122

dakkar added a commit to dakkar/Sharkey-thenautilus that referenced this issue Oct 31, 2023
`note.fileIds` is an array, it's always going to be truthy; we need to
check the length, like a similar `if` does just a few lines above
@Mar0xy Mar0xy closed this as completed in 111863b Oct 31, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants