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

Buddypress connector implemented #1211

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

kidunot89
Copy link
Contributor

@kidunot89 kidunot89 commented Dec 23, 2020

Partially resolves #1093

Summary checklist

  • activity callbacks tested and passing.
  • group callbacks tested and passing.
  • option callbacks tested and passing.
  • xprofile callbacks tested and passing.
  • callback_xprofile_group_before_save and callback_xprofile_field_before_save callbacks added to the BuddyPress connector class to provide a more stable method of capturing the incoming field/field group action that doesn't rely on $wpdb->insert_id.

public function callback_xprofile_field_before_save( $field ) {
$this->incoming_field_action = empty( $field->id ) ? 'created' : 'updated';
}

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ivankruchkoff Set my comment below

}

$action = $this->incoming_field_group_action;
unset( $this->incomming_field_group_action );
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ivankruchkoff I below this is a better way to capture the incoming field action

@kidunot89 kidunot89 added this to the 3.7.0 milestone Jan 4, 2021
@kidunot89 kidunot89 force-pushed the tests/buddypress-connector branch from e02d472 to 5309c3f Compare February 19, 2021 23:58
@kidunot89 kidunot89 changed the base branch from develop to master February 20, 2021 00:02
@kidunot89 kidunot89 changed the base branch from master to develop February 20, 2021 00:03
@kidunot89 kidunot89 mentioned this pull request Feb 22, 2021
7 tasks
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.

Improve Test Coverage
1 participant