[samuelsov]

This is a follow-up on:

• Extractor for SK managed file which have implicit ts #25
• Multilingual SK: extracting strings to civicrm_translation_source civicrm-core#35795
• Remove ts from savedSearch settings strings civicrm-core#35860

Add to mgd.php extractor empty_value and rewrite support as per https://github.com/civicrm/civicrm-core/pull/35860/changes#discussion_r3356537282

Also, the isWorthy is updated to be more restrictive the same way we do in https://github.com/civicrm/civicrm-core/pull/35795/changes#diff-28f92e8856eeed4876a3db8314849540334b78d799fcce9411d2be34a021b51fR84

[eileenmcnaughton]

@samuelsov is this one that @mlutfy is reviewing?

[samuelsov]

After discussing with @mlutfy, it seems that the test for html type is rather unnecessary as we we want to be able to translate pure html like:

<h2>This is my <b>important title</b></h2>

The example given by @colemanw is already excluded from translation because it contains {ts}
https://github.com/civicrm/civicrm-core/pull/35860/changes#diff-d24b221e2b30a568dc91748b1b2dd42a3f6bf09c5b85d2701e31651c232954d0R88

'rewrite' => '<b>{ts}Summary:{/ts}</b> [description]<br><b>{ts}Details:{/ts}</b><pre>[data]</pre>',

So it's probably better to remove the extra check to exclude rewrite if type=html.

[samuelsov]

@mlutfy Fixed as discussed and added a small fix by coleman on isWorthy. So it should be good to go.