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

Bug in Attachment Manager #9729

Closed
rich20 opened this issue Sep 2, 2024 · 6 comments
Closed

Bug in Attachment Manager #9729

rich20 opened this issue Sep 2, 2024 · 6 comments

Comments

@rich20
Copy link
Member

rich20 commented Sep 2, 2024

Describe the bug
https://www.kunena.org/forum/k-6-3-0-support/168551-huge-bug-in-attachment-manager
Confirmed

To Reproduce

  1. Write a text and add an attachment into the message
  2. Remove this attachment via Attachment Manager

Result
It removes the attachment together with the text and places the number 1 instead

System information

  • Joomla version: ‎5.1.5-dev
  • Joomla template: CASSIOPEIA
  • Kunena version: 6.3.5-DEV
  • Kunena template: Aurelia
  • Php version: 8.2.20
  • Database version: 10.5.25-MariaDB

Desktop

  • OS: Ubuntu 22.04.1 LTS
  • Browser: Chromium, Firefox and Opera
@rich20 rich20 added this to the 6.2 milestone Sep 2, 2024
@rich20 rich20 added 6.3 and removed 6.2 labels Sep 2, 2024
@rich20 rich20 modified the milestones: 6.2, 6.3 Sep 2, 2024
@RakhmetKotlin
Copy link

Hi. This problem also occurs when you remove attachment via Kunena Backend.

@Pinkeltje
Copy link
Contributor

Just my five cents, but I think at least message text isn't removed when changing line 1316 to
$message_text = $attachment->removeBBCodeInMessage($editor_text);
Problem remains that bbcode isn't removed.
2024-09-04 Screenshot 570
2024-09-04 Screenshot 571

Copy link

github-actions bot commented Oct 7, 2024

This issue is stale because it has been open 30 days with no activity.

@xillibit
Copy link
Member

Result
It removes the attachment together with the text and places the number 1 instead

Fixed in : 27825ec

Copy link

This issue is stale because it has been open 30 days with no activity.

@rich20
Copy link
Member Author

rich20 commented Nov 22, 2024

Is already solved.

@rich20 rich20 closed this as completed Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants