[wp-meta] [Making WordPress.org] #5117: WP dashboard city locations (encoding?) wrong

Making WordPress.org noreply at wordpress.org
Sun Mar 29 01:47:49 UTC 2020


#5117: WP dashboard city locations (encoding?) wrong
-----------------------------+---------------------
 Reporter:  ramon fincken    |       Owner:  (none)
     Type:  defect           |      Status:  new
 Priority:  normal           |   Milestone:
Component:  Events API       |  Resolution:
 Keywords:  has-screenshots  |
-----------------------------+---------------------
Changes (by dd32):

 * keywords:  has-screenshots needs-patch => has-screenshots
 * milestone:  Q2 =>


Comment:

 > I'm not 100% sure on this, but I think this happens because the table
 has the latin1_general_ci collation set for it. Think the table needs a
 restructure to be fully utf8mb4 for all rows and the collation.

 The table collation is set to latin1 (as all of WordPress.org should be)
 but the data is utf8 and treated as such since the connection to the table
 should be utf8 (When using not-the-events-api to query it, that needs to
 be set manually)

 It looks like this is a MySQL ordering issue, adding an exact-match in the
 ordering seems to result in the correct row being returned, I suspect this
 wasn't picked up at the time of building since the row orderings on disk
 in the table have since changed, previously the first result would've been
 the correct, but now the correct row is a few results in.

 Commit incoming, the Caches on location lookups is 12hours, so if everyone
 could recheck the API responses after that time, it'd be appreciated to
 check that everything is still responding as intended. That goes for more
 than just Zurich/Amsterdam, "normal" things that were previously correct
 should also be checked. Keep in mind that the WordPress Timezone and
 Locale are also taken into account, so even if it was working fine for
 someone, double check it.

-- 
Ticket URL: <https://meta.trac.wordpress.org/ticket/5117#comment:5>
Making WordPress.org <https://meta.trac.wordpress.org/>
Making WordPress.org


More information about the wp-meta mailing list