Skip to:
Content

BuddyPress.org

Opened 2 years ago

Closed 19 months ago

#9092 closed defect (bug) (fixed)

Member type have no default title

Reported by: 1naveengiri's profile 1naveengiri Owned by: espellcaste's profile espellcaste
Milestone: 15.0.0 Priority: normal
Severity: normal Version: 7.0.0
Component: Core Keywords: has-patch dev-reviewed
Cc:

Description

I have added a member type with just Slug and no name.
https://ibb.co/dPYLtG2

After this, I found an empty checkbox with no default title.
https://ibb.co/XLx1jn9

In my opinion, it should either say (no name) or it should use the member type slug as the name of the membership type.


Attachments (10)

Screenshot 2024-02-10 at 10.03.33 PM.png (223.5 KB) - added by 1naveengiri 2 years ago.
member type with no name
Screenshot 2024-02-10 at 10.02.48 PM.png (230.6 KB) - added by 1naveengiri 2 years ago.
Empty checkbox for member type
member-type-required.png (54.0 KB) - added by imranmd 2 years ago.
Add required label on all three fileds.
singular-name-required-message.png (59.3 KB) - added by imranmd 2 years ago.
Add message for singular name required field.
plural-name-required-message.png (59.0 KB) - added by imranmd 2 years ago.
Add message for plural name required field.
9092.patch (4.1 KB) - added by imranmd 2 years ago.
patch added
group-type-required.png (58.0 KB) - added by imranmd 2 years ago.
Add required label on all three fields for group.
group-singular-name-required-message.png (64.1 KB) - added by imranmd 2 years ago.
Add message for singular name required field for group.
group-plural-name-required-message.png (63.6 KB) - added by imranmd 2 years ago.
Add message for plural name required field for group.
9092.2.patch (5.6 KB) - added by imranmd 2 years ago.
patch added

Download all attachments as: .zip

Change History (23)

@1naveengiri
2 years ago

member type with no name

@1naveengiri
2 years ago

Empty checkbox for member type

#1 @imath
2 years ago

  • Component changed from Extended Profile to Members
  • Milestone changed from Awaiting Review to 14.0.0
  • Version changed from 12.2.0 to 7.0.0

Hi @1naveengiri

Thanks for you report. We'll fix it soon.

@imranmd
2 years ago

Add required label on all three fileds.

@imranmd
2 years ago

Add message for singular name required field.

@imranmd
2 years ago

Add message for plural name required field.

@imranmd
2 years ago

patch added

#2 @imranmd
2 years ago

  • Keywords needs-patch removed

@imath I have checked above mentioned issue. I made the singular and plural name fields required to resolve this issue and added patch. Also, I have attached screenshot after resolved this issue.

#3 @imath
2 years ago

  • Keywords has-patch added

Thanks e lot for your patch @imranmd

I believe some functions are used for both Group and Member types. This one in particular: bp_core_admin_insert_type(). That's probably the reason why "Type" was used instead of "Member type". Could you test adding a group type to see if your patch doesn't introduce a side effect there?

@imranmd
2 years ago

Add required label on all three fields for group.

@imranmd
2 years ago

Add message for singular name required field for group.

@imranmd
2 years ago

Add message for plural name required field for group.

@imranmd
2 years ago

patch added

#4 @imranmd
2 years ago

Hi @imath,

Apologies for the oversight. I initially checked only for the member type. However, in my new patch, I've included code for both member and group types.

This ticket was mentioned in Slack in #buddypress by espellcaste. View the logs.


2 years ago

#6 @espellcaste
2 years ago

  • Milestone changed from 14.0.0 to Up Next
  • Owner set to espellcaste
  • Status changed from new to assigned

#7 @imath
20 months ago

  • Milestone changed from Up Next to 15.0.0

#8 @espellcaste
20 months ago

  • Component changed from Members to Core

@imranmd I can confirm your bug report. I'll test your suggested patch.

#9 @imranmd
20 months ago

Thank you @espellcaste

#11 @espellcaste
19 months ago

  • Keywords dev-reviewed added

Tested the patch. Good addition. Added here: https://github.com/buddypress/buddypress/pull/335

#12 @imranmd
19 months ago

Thanks @espellcaste

#13 @espellcaste
19 months ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 13983:

Require singular and plural names for BuddyPress custom types.

When creating a member or group type, require that both singular and plural names are provided. This ensures that the type can be properly registered and used throughout BuddyPress.

Props 1naveengiri, imath, imranmd.

Fixes #9092
Closes https://github.com/buddypress/buddypress/pull/335/

Note: See TracTickets for help on using tickets.