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

Relationship Field improperly connecting to data taken from the database when WordPress is in multiple network mode #7181

Closed
ntinhhcm opened this issue Sep 23, 2023 · 1 comment · Fixed by #7185
Assignees
Labels
Focus: Other Compatibility Compatibility with third-party plugins, themes, and libraries Status: Has Patch Has a linked PR Type: Bug
Milestone

Comments

@ntinhhcm
Copy link

ntinhhcm commented Sep 23, 2023

Description

In the pods/classes/fields/picks.php at line 2740

Replace

$prefix = $wpdb->base_prefix;

if ( is_multisite() && ! is_main_site() ) {
    $prefix .= get_current_blog_id() . '_';
}

with

$prefix = $wpdb->prefix;

// if ( is_multisite() && ! is_main_site() ) {
//     $prefix .= get_current_blog_id() . '_';
// }

When we want to get the list of users on the current site, the result returns the list of the original site.

Version

3.0.3

Testing Instructions

No response

Screenshots / Screencast

No response

Possible Workaround

No response

Site Health Information

No response

Pods Package

No response

JoryHogeveen added a commit that referenced this issue Sep 25, 2023
Enhance compatibility with other plugins.
Fixes #7181
@JoryHogeveen JoryHogeveen added the Focus: Other Compatibility Compatibility with third-party plugins, themes, and libraries label Sep 25, 2023
@JoryHogeveen JoryHogeveen self-assigned this Sep 25, 2023
@JoryHogeveen JoryHogeveen added this to the Pods 3.0.4 milestone Sep 25, 2023
@JoryHogeveen JoryHogeveen added the Status: Has Patch Has a linked PR label Sep 25, 2023
@sc0ttkclark sc0ttkclark modified the milestones: Pods 3.0.4, Pods 3.0.5 Sep 28, 2023
@sc0ttkclark
Copy link
Member

Fixed via #7185

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Focus: Other Compatibility Compatibility with third-party plugins, themes, and libraries Status: Has Patch Has a linked PR Type: Bug
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants