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

(NFC) Fix typo in Money valueFormat depretation warning #18886

Merged
merged 1 commit into from
Oct 30, 2020

Conversation

mlutfy
Copy link
Member

@mlutfy mlutfy commented Oct 30, 2020

Overview

This is pretty silly, but I stumbled on this warning on a site (civicrm.org!) and as a fix, I copy-pasted the format recommended in the deprecation warning, but it has a typo, which caused a formatting error.

  • Before: !%i
  • After: %!i

I'm not sure why it was complaining, since it was correctly set before:

MariaDB [civicrmorg]> select * from log_civicrm_setting where name = 'moneyvalueformat' order by log_date desc limit 100;
+------+------------------+-------------+-----------+------------+-----------+--------------+---------------------+------------+---------------------+---------------+-------------+----------------+
| id   | name             | value       | domain_id | contact_id | is_domain | component_id | created_date        | created_id | log_date            | log_conn_id   | log_user_id | log_action     |
+------+------------------+-------------+-----------+------------+-----------+--------------+---------------------+------------+---------------------+---------------+-------------+----------------+
|  568 | moneyvalueformat | s:3:"%!i";  |         1 |       NULL |         1 |         NULL | 2020-10-30 09:04:59 |       3039 | 2020-10-30 13:04:59 | 5f9c0f7adcb9b |        3039 | Update         |
|  568 | moneyvalueformat | s:4:".%!i"; -> me testing |         1 |       NULL |         1 |         NULL | 2020-10-30 09:03:52 |       3039 | 2020-10-30 13:03:52 | 5f9c0f37ec378 |        3039 | Update         |
|  568 | moneyvalueformat | s:3:"%!i";  |         1 |       NULL |         1 |         NULL | 2020-10-30 09:02:32 |       3039 | 2020-10-30 13:02:32 | 5f9c0ee8470fe |        3039 | Update         |
|  568 | moneyvalueformat | s:3:"!%i";  -> my bad |         1 |       NULL |         1 |         NULL | 2020-10-29 16:53:06 |       3039 | 2020-10-29 20:53:06 | 5f9b2bb1cd9b3 |        3039 | Update         |
|  568 | moneyvalueformat | s:3:"%!i";  |         1 |       NULL |         1 |         NULL | 2020-04-14 09:53:57 |       3039 | 2020-04-14 13:53:57 | 5e95c074e1b07 |        3039 | Update         |
|  568 | moneyvalueformat | s:3:"%!i";  |         1 |       NULL |         1 |         NULL | 2020-04-14 09:51:47 |       3039 | 2020-04-14 13:51:47 | 5e95bff3acbb5 |        3039 | Update         |
|  568 | moneyvalueformat | s:3:"%!i";  |         1 |       NULL |         1 |         NULL | 2020-03-26 11:10:31 |       3039 | 2020-03-26 15:10:31 | 5e7cc5e761958 |        3039 | Update         |
|  568 | moneyvalueformat | s:3:"%!i";  |         1 |       NULL |         1 |         NULL | 2019-09-02 14:43:58 |       3039 | 2019-09-02 18:43:58 | 5d6d62ee42391 |        3039 | Update         |

@civibot
Copy link

civibot bot commented Oct 30, 2020

(Standard links)

@civibot civibot bot added the master label Oct 30, 2020
@seamuslee001
Copy link
Contributor

looks good merging

@seamuslee001 seamuslee001 merged commit 75f113a into civicrm:master Oct 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants