← All changes
|
includes/abilities/settings/class-update-image-seo-settings.php
+9
-2
2.0.1
→
2.8.0
View file →
| @@ -47,9 +47,9 @@ | ||
| 47 | 47 | */ |
| 48 | 48 | public function __construct() { |
| 49 | 49 | $this->id = 'thinkrank/update-image-seo-settings'; |
| 50 | 50 | $this->label = __( 'Update ThinkRank Image SEO Settings', 'thinkrank' ); |
| 51 | - $this->description = __( 'Update ThinkRank image SEO settings, which control automatic alt/title injection, the token-based formats used to generate them, and whether generated alt text is persisted to the Media Library.', 'thinkrank' ); | |
| 51 | + $this->description = __( 'Update ThinkRank image SEO settings, which control automatic alt/title injection, the token-based formats used to generate them, and whether generated alt text is persisted to the Media Library. Read the current values with get-image-seo-settings first; only the keys you pass are changed.', 'thinkrank' ); | |
| 52 | 52 | } |
| 53 | 53 | |
| 54 | 54 | /** |
| 55 | 55 | * {@inheritDoc} |
| @@ -58,9 +58,16 @@ | ||
| 58 | 58 | */ |
| 59 | 59 | public function get_annotations() { |
| 60 | 60 | return [ |
| 61 | 61 | 'readonly' => false, |
| 62 | - 'destructive' => true, | |
| 62 | + // Settings writes are recoverable: the matching get-* ability reads | |
| 63 | + // the previous value, so nothing is lost that cannot be put back. | |
| 64 | + // `destructive` is reserved for calls that lose data or reach | |
| 65 | + // outside the site, and marking routine configuration with it made | |
| 66 | + // MCP clients demand a human approval for every save — which users | |
| 67 | + // reported as a permission bug, because the client's refusal reads | |
| 68 | + // as "No approval received" (#675). | |
| 69 | + 'destructive' => false, | |
| 63 | 70 | 'idempotent' => true, |
| 64 | 71 | 'priority' => 2.0, |
| 65 | 72 | 'openWorldHint' => false, |
| 66 | 73 | ]; |