v4 Error with Pro

Resolved nimonogi
(@nimonogi)

3 years ago

After upgrading my Pro plugin to v4, I am getting the following error every time i try to sent an sms from the order’s page:

Can’t resolve the customer mobile number.

Viewing 11 replies - 1 through 11 (of 11 total)

Plugin Author Mostafa Soufi
(@mostafas1990)

3 years ago

Hello,

Thank you for opening the topic, can you please double check what option is set for SMS → Settings → General → Mobile field status?

Thread Starter nimonogi
(@nimonogi)

3 years ago

“Use the existing phone field” is selected.

ps. Just to note that everything was working fine until upgraded to v4.

Plugin Author Mostafa Soufi
(@mostafas1990)

3 years ago

So, can you please change it to “Use the exist phone field in billing address” and then try again?
Thread Starter nimonogi
(@nimonogi)

3 years ago
This is what is already selected as I said.
- This reply was modified 3 years ago by nimonogi.
Plugin Author Mostafa Soufi
(@mostafas1990)

3 years ago
In this case we should make sure in what field the mobile numbers are stored, can you look at one order on database to make sure what is the field key?

an example of query:
```
SELECT * FROM wp_postmeta WHERE meta_value LIKE 'mobilenumber'
```
- This reply was modified 3 years ago by Mostafa Soufi.
Thread Starter nimonogi
(@nimonogi)

3 years ago

MySQL returned an empty result set (i.e. zero rows).

Apparently the problem rises from the last update. Some changes you made there doesn’t work properly as I were using the plugin Everyday for more than a year now.

Plugin Author Mostafa Soufi
(@mostafas1990)

3 years ago

Btw, did you replace the mobilenumber on query to a real customer’s number?

Thread Starter nimonogi
(@nimonogi)

3 years ago

Sorry, here are the proper results: https://imgur.com/a/OcSHWIs

Plugin Author Mostafa Soufi
(@mostafas1990)

3 years ago

Thank you, now is clear and I fixed it, so can you please test it?

https://github.com/veronalabs/wp-sms/commit/e0c122e7e8501250b1e61520ffc5c0c3861d105c#diff-5f74479e42efce4c0ed16d7de0695f6ba65645e91b79c0eb5ac6b65af8d38ab4

https://github.com/veronalabs/wp-sms/commit/f262082f8a9db5420ed5e8003f81d0f6b8b96635

I’ll prepare to release the new version once you confirmed.

Thread Starter nimonogi
(@nimonogi)

3 years ago

Thanks, that seems to resolving the problem.

Plugin Author Mostafa Soufi
(@mostafas1990)

3 years ago

Sounds good, thank you for testing, I’ll release the new version soon.

Viewing 11 replies - 1 through 11 (of 11 total)

The topic ‘v4 Error with Pro’ is closed to new replies.