Plugin Author
axew3
(@axewww)
OK! Look
= 2.7.1 =
Release Date – 26 Apr, 2023
- Fix: wp_insert_user function, used by the ‘phpBB extension’ (if it has been installed into phpBB, and option active) for the user addition in WP via cURL, when an user register in phpBB side: on tests it fail with error ‘Not enough data provided’ when the email value is provided (WP 6.2)
- Minor fixes
Thank you for the info, yes i assume it is like the above. The plugin code has been tested into latest Php 8.2.6 and will be released very soon. Going to fix the transfer processes (all) that are affected by the same problem!
Mhhh
add_filter( ‘wp_pre_insert_user_data’, ‘w3all_wp_pre_insert_user_data’, 10, 4 );
removed….
it has been fixed by me instead last time, not providing an email, on file wp_w3all.php
function w3all_add_phpbb_user() {
the email value is not passed, then the user’s table is going to be updated (adding then the email).
//'user_email' => $phpbb_user[0]->user_email, // on WP 6.2 the wp_insert_user function, on cULR tests, fail with error "Not enough data provided" when email value provided
You say that disabling the hook wp_pre_insert_user_data it work instead. Ok let check the whole think and if you have any news let know!
See you soon, really thank to you!
-
This reply was modified 2 years, 10 months ago by
axew3.
-
This reply was modified 2 years, 10 months ago by
axew3.
Plugin Author
axew3
(@axewww)
i assume that the function w3all_wp_pre_insert_user_data into wp_w3all.php file:
it should be
return $userdata;
and not the actual
return $data;
anyway i did not get actually any error into php 8.2.6 and latest wp, both when inserting or creating a new user.
Ps wrong, checked that it need to be return $data
i will check further more the thing
-
This reply was modified 2 years, 10 months ago by
axew3.
Plugin Author
axew3
(@axewww)
I have try out testing, that i only get
Error: username or email already exists into our forum.
Return back
error message from the plugin, when an username or email already exists into the phpbb db. At moment i cannot reproduce the issue, but looking to change the hooked function so to not interfere in any way, if/when the email/username is not found into phpbb
Plugin Author
axew3
(@axewww)
nope… it seem to me an error due to something wrong in your WP or something else in case, it seem that already it do not interfere in any way if the email/username do not exists into phpBB.
If you imagine any other hint to reverse your problem or fix the thing please let know! I still will follow on check the issue by the way
-
This reply was modified 2 years, 10 months ago by
axew3.
Plugin Author
axew3
(@axewww)
the hook call will be wrapped with
if ( ! defined( ‘WP_ADMIN’ ) )
{
}
so that in effect, since it is a trick for front end plugins pages (it is NOT necessary for WP default flow, where another check is done and it is sufficient), it will be fired only when registrations fires on front end using external plugins. It should be redundant to do these check (if email exist into phpBB), and should be may removed, because, once the integration run with paired users, if an email exist wp himself will avoid the registration of the existent user. Further more, using the phpBB WP extension in phpBB, it is possible to leave users to register and update profiles (email) both in wp and phpBB.
The logic should say: remove it. But ok, in the while we’ll wrap the hook to fire only when on frontend plugins pages.
-
This reply was modified 2 years, 10 months ago by
axew3.
Plugin Author
axew3
(@axewww)
2.7.2 has been just released