Skip to content

Update a test about bp_attachments_get_allowed_types()#401

Closed
imath wants to merge 1 commit intobuddypress:masterfrom
imath:patch/9251
Closed

Update a test about bp_attachments_get_allowed_types()#401
imath wants to merge 1 commit intobuddypress:masterfrom
imath:patch/9251

Conversation

@imath
Copy link
Copy Markdown
Contributor

@imath imath commented Oct 31, 2024

Test document types instead of image ones as BP image supported types are not the same than WordPress.

Trac ticket: https://buddypress.trac.wordpress.org/ticket/9251


This Pull Request is for code review only. Please keep all other discussion in the BuddyPress Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the WordPress Core Handbook for more details.

Test `document` types instead of `image` ones as BP image supported types are not the same than WordPress.
@imath imath deleted the patch/9251 branch October 31, 2024 05:18
dcavins pushed a commit to dcavins/buddypress-wp-svn that referenced this pull request Nov 3, 2024
Use a more meaningful test checking for `document` file types.

Fixes #9251
Closes buddypress/buddypress#401



git-svn-id: http://buddypress.svn.wordpress.org/trunk@14067 cdf35c40-ae34-48e0-9cc9-0c9da1808c22
emaralive pushed a commit to emaralive/omt-buddypress-sync that referenced this pull request Nov 2, 2025
Use a more meaningful test checking for `document` file types.

Fixes #9251
Closes buddypress/buddypress#401



git-svn-id: https://buddypress.svn.wordpress.org/trunk@14067 cdf35c40-ae34-48e0-9cc9-0c9da1808c22
emaralive pushed a commit to emaralive/buddypress-sync that referenced this pull request Nov 3, 2025
Use a more meaningful test checking for `document` file types.

Fixes #9251
Closes buddypress/buddypress#401



git-svn-id: https://buddypress.svn.wordpress.org/trunk@14067 cdf35c40-ae34-48e0-9cc9-0c9da1808c22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant