← All changes
|
includes/abilities/settings/class-update-schema-settings.php
+9
-2
2.0.2
→
2.7.0
View file →
| @@ -103,9 +103,9 @@ | ||
| 103 | 103 | */ |
| 104 | 104 | public function __construct() { |
| 105 | 105 | $this->id = 'thinkrank/update-schema-settings'; |
| 106 | 106 | $this->label = __( 'Update ThinkRank Schema Settings', 'thinkrank' ); |
| 107 | - $this->description = __( 'Update ThinkRank structured-data (schema) settings. When auto_deploy is enabled, saving automatically regenerates and redeploys the site-wide schema markup.', 'thinkrank' ); | |
| 107 | + $this->description = __( 'Update ThinkRank structured-data (schema) settings. When auto_deploy is enabled, saving automatically regenerates and redeploys the site-wide schema markup. Read the current values with get-schema-settings first; only the keys you pass are changed.', 'thinkrank' ); | |
| 108 | 108 | } |
| 109 | 109 | |
| 110 | 110 | /** |
| 111 | 111 | * {@inheritDoc} |
| @@ -114,9 +114,16 @@ | ||
| 114 | 114 | */ |
| 115 | 115 | public function get_annotations() { |
| 116 | 116 | return [ |
| 117 | 117 | 'readonly' => false, |
| 118 | - 'destructive' => true, | |
| 118 | + // Settings writes are recoverable: the matching get-* ability reads | |
| 119 | + // the previous value, so nothing is lost that cannot be put back. | |
| 120 | + // `destructive` is reserved for calls that lose data or reach | |
| 121 | + // outside the site, and marking routine configuration with it made | |
| 122 | + // MCP clients demand a human approval for every save — which users | |
| 123 | + // reported as a permission bug, because the client's refusal reads | |
| 124 | + // as "No approval received" (#675). | |
| 125 | + 'destructive' => false, | |
| 119 | 126 | 'idempotent' => true, |
| 120 | 127 | 'priority' => 2.0, |
| 121 | 128 | 'openWorldHint' => false, |
| 122 | 129 | ]; |