← All changes | include/mo.php +21 -35 2.7.2 → 2.0.3 View file →

		@@ -1,16 +1,15 @@
1	1	<?php
2	2
3	3	/**
4		- * Manages strings translations storage
	4	+ * manages strings translations storage
5	5	*
6	6	* @since 1.2
7		- * @since 2.1 Stores the strings in a post meta instead of post content to avoid unserialize issues (See #63)
8	7	*/
9	8	class PLL_MO extends MO {
10	9
11	10	/**
12		- * Registers the polylang_mo custom post type, only at first object creation
	11	+ * registers the polylang_mo custom post type, only at first object creation
13	12	*
14	13	* @since 1.2
15	14	*/
16	15	public function __construct() {
		@@ -16,55 +15,51 @@
16	15	public function __construct() {
17	16	if ( ! post_type_exists( 'polylang_mo' ) ) {
18	17	$labels = array( 'name' => __( 'Strings translations', 'polylang' ) );
19	18	register_post_type( 'polylang_mo', array( 'labels' => $labels, 'rewrite' => false, 'query_var' => false, '_pll' => true ) );
20		-
21		- add_action( 'pll_add_language', array( $this, 'clean_cache' ) );
22	19	}
23	20	}
24	21
25	22	/**
26		- * Writes a PLL_MO object into a custom post meta
	23	+ * writes a PLL_MO object into a custom post
27	24	*
28	25	* @since 1.2
29	26	*
30		- * @param object $lang The language in which we want to export strings
	27	+ * @param object $lang the language in which we want to export strings
31	28	*/
32	29	public function export_to_db( $lang ) {
33		- $this->add_entry( $this->make_entry( '', '' ) ); // Empty string translation, just in case
	30	+ $this->add_entry( $this->make_entry( '', '' ) ); // empty string translation, just in case
34	31
35		- // Would be convenient to store the whole object but it would take a huge space in DB
36		- // So let's keep only the strings in an array
	32	+ // would be convenient to store the whole object but it would take a huge space in DB
	33	+ // so let's keep only the strings in an array
37	34	$strings = array();
38	35	foreach ( $this->entries as $entry ) {
39	36	$strings[] = array( $entry->singular, $this->translate( $entry->singular ) );
40	37	}
41	38
42		- $strings = wp_slash( $strings ); // Avoid breaking slashed strings in update_post_meta. See https://codex.wordpress.org/Function_Reference/update_post_meta#Character_Escaping
	39	+ // we need to make sure that $post is empty when $lang->mo_id is empty: see https://wordpress.org/support/topic/problem-when-adding-a-language
	40	+ $post = empty( $lang->mo_id ) ? array() : get_post( $lang->mo_id, ARRAY_A ); // wp_insert_post wants an array
43	41
44		- if ( empty( $lang->mo_id ) ) {
45		- $post = array(
46		- 'post_title' => 'polylang_mo_' . $lang->term_id,
47		- 'post_status' => 'private', // To avoid a conflict with WP Super Cache. See https://wordpress.org/support/topic/polylang_mo-and-404s-take-2
48		- 'post_type' => 'polylang_mo',
49		- );
50		- $mo_id = wp_insert_post( $post );
51		- update_post_meta( $mo_id, '_pll_strings_translations', $strings );
52		- } else {
53		- update_post_meta( $lang->mo_id, '_pll_strings_translations', $strings );
54		- }
	42	+ $post['post_title'] = 'polylang_mo_' . $lang->term_id;
	43	+ // json_encode would take less space but is slower to decode
	44	+ // wp_insert_post expects slashed data
	45	+ $post['post_content'] = addslashes( serialize( $strings ) );
	46	+ $post['post_status'] = 'private'; // to avoid a conflict with WP Super Cache. See https://wordpress.org/support/topic/polylang_mo-and-404s-take-2
	47	+ $post['post_type'] = 'polylang_mo';
	48	+ wp_insert_post( $post );
55	49	}
56	50
57	51	/**
58		- * Reads a PLL_MO object from a custom post meta
	52	+ * reads a PLL_MO object from a custom post
59	53	*
60	54	* @since 1.2
61	55	*
62		- * @param object $lang The language in which we want to get strings
	56	+ * @param object $lang the language in which we want to get strings
63	57	*/
64	58	public function import_from_db( $lang ) {
65	59	if ( ! empty( $lang->mo_id ) ) {
66		- $strings = get_post_meta( $lang->mo_id, '_pll_strings_translations', true );
	60	+ $post = get_post( $lang->mo_id, OBJECT );
	61	+ $strings = unserialize( $post->post_content );
67	62	if ( is_array( $strings ) ) {
68	63	foreach ( $strings as $msg ) {
69	64	$this->add_entry( $this->make_entry( $msg[0], $msg[1] ) );
70	65	}
		@@ -72,9 +67,9 @@
72	67	}
73	68	}
74	69
75	70	/**
76		- * Returns the post id of the post storing the strings translations
	71	+ * returns the post id of the post storing the strings translations
77	72	*
78	73	* @since 1.4
79	74	*
80	75	* @param object $lang
		@@ -91,15 +86,6 @@
91	86	}
92	87
93	88	// The mo id for a language can be transiently empty
94	89	return isset( $ids[ 'polylang_mo_' . $lang->term_id ] ) ? $ids[ 'polylang_mo_' . $lang->term_id ]->ID : null;
95		- }
96		-
97		- /**
98		- * Invalidate the cache when adding a new language
99		- *
100		- * @since 2.0.5
101		- */
102		- public function clean_cache() {
103		- wp_cache_delete( 'polylang_mo_ids' );
104	90	}
105	91	}