Docs: Improve various globals documentation, as per docblock standards.

[upadalavipul]

Function commenting details needs to improve.

Files:

• src/bp-xprofile/classes/class-bp-xprofile-group.php
• src/bp-xprofile/classes/class-bp-xprofile-meta-query.php
• src/bp-xprofile/classes/class-bp-xprofile-profiledata.php
• src/bp-xprofile/classes/class-bp-xprofile-query.php

[imath]

Hi @upadalavipul

Thanks for your contribution, patch looks good 👍. I will commit it shortly.

[imath]

In 13475:

Improve inline doc, informing about $wpdb usage in functions

Props upadalavipul

Fixes #8890

[upadalavipul]

Few more files in improve various globals documentation.

[imath]

In 13484:

Improve inline doc, informing about global variable usage in functions

In various xProfile component files, used globals are:

• $wpdb: the WordPress database object,
• $profile_template: the xProfile data template object,
• $group: the current group of profile fields in a loop,
• $field: the current field in a loop.

Props upadalavipul

Fixes #8890

[viralsampat]

I have checked above mentioned issue and founds few files. Here, I have added its patch.

[viralsampat]

I have checked above mentioned issue and founds few files. Here, I have added its patch.

[viralsampat]

I have checked above mentioned issue and founds few files. Here, I have added its patch.

[imath]

Hi @viralsampat

Thanks a lot for these 3 new patches. I'll review it asap.

[imath]

Hi @viralsampat

For your next contributions, could you use the --no-prefix option when saving your diffs. Thanks in advance 🙏.

I'll commit the inline doc improvements shortly.

[imath]

In 13491:

Improve inline doc, informing about global variable usage in functions

Props viralsampat

Fixes #8890

[viralsampat]

I have checked above mentioned issue and founds few files. Here, I have added its patch.

[viralsampat]

I have checked above mentioned issue and founds few files. Here, I have added its patch.

[imath]

Hi thanks for the patch @viralsampat as I've requested in comment:6 using the --no-prefix option would have save us some more time, please consider using it for your next patches. By the way, you can include more than one file into your patches.

[imath]

Also, please use the ​BuddyPress trunk version to build your diffs, bp_core_set_uri_globals() is deprecated and is no more in src/bp-core/bp-core-catchuri.php meaning ​this patch is outdated.

[viralsampat]

I have checked above mentioned issue and founds few files. Here, I have added its patch.

[imath]

Thanks for the refresh I will check the patch and commit changes asap.

[imath]

In 13496:

BP Groups: inform about global variable usage in functions docblocks

Props viralsampat

Fixes #8890

[viralsampat]

I have checked above mentioned issue and founds few files. Here, I have added its patch.

[imath]

Hi @viralsampat Thanks a lot for this new patch. I will review & commit it asap.

[viralsampat]

I have checked above mentioned issue and founds few files. Here, I have added its patch.

[viralsampat]

I have checked above mentioned issue and founds few files. Here, I have added its patch.

[imath]

In 13499:

Improve inline doc, informing about global variable usage in functions

These improvements are about various BP Members, BP Messages & BP Notifications files.

Props viralsampat

Fixes #8890

[upadalavipul]

Few more files in improve various globals documentation.

[imath]

In 13517:

Inform about $activities_template usage in some functions inline doc

Props upadalavipul

Fixes #8890