Skip to content

Commit 5882e60

Browse files
committed
Address Copilot review feedback
- Move load_plugin_textdomain to init hook for proper timing - Add priority 9 to integration hooks to maintain original behavior
1 parent d3e4519 commit 5882e60

1 file changed

Lines changed: 15 additions & 8 deletions

File tree

includes/class-nodeinfo.php

Lines changed: 15 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -73,26 +73,33 @@ public function init() {
7373
$this->register_hooks();
7474
$this->register_admin_hooks();
7575

76-
// Load language files.
76+
$this->initialized = true;
77+
}
78+
79+
/**
80+
* Load the plugin text domain.
81+
*/
82+
public function load_textdomain() {
7783
\load_plugin_textdomain(
7884
self::TEXT_DOMAIN,
7985
false,
8086
\dirname( \plugin_basename( NODEINFO_PLUGIN_FILE ) ) . '/languages'
8187
);
82-
83-
$this->initialized = true;
8488
}
8589

8690
/**
8791
* Register hooks.
8892
*/
8993
public function register_hooks() {
94+
// Load plugin text domain.
95+
\add_action( 'init', array( $this, 'load_textdomain' ) );
96+
9097
// Initialize NodeInfo version integrations.
91-
\add_action( 'init', array( Nodeinfo10::class, 'init' ) );
92-
\add_action( 'init', array( Nodeinfo11::class, 'init' ) );
93-
\add_action( 'init', array( Nodeinfo20::class, 'init' ) );
94-
\add_action( 'init', array( Nodeinfo21::class, 'init' ) );
95-
\add_action( 'init', array( Nodeinfo22::class, 'init' ) );
98+
\add_action( 'init', array( Nodeinfo10::class, 'init' ), 9 );
99+
\add_action( 'init', array( Nodeinfo11::class, 'init' ), 9 );
100+
\add_action( 'init', array( Nodeinfo20::class, 'init' ), 9 );
101+
\add_action( 'init', array( Nodeinfo21::class, 'init' ), 9 );
102+
\add_action( 'init', array( Nodeinfo22::class, 'init' ), 9 );
96103

97104
// Register REST routes.
98105
\add_action( 'rest_api_init', array( $this, 'register_routes' ) );

0 commit comments

Comments
 (0)