Revert "fix(compiler): xmb `<ph>` tags should not self close (#13413)"

This reverts commit 4b3d135193.
closes #13463
This commit is contained in:
Victor Berchet 2016-12-14 10:12:13 -08:00
parent 8c27c62fab
commit f238c8ac7a
4 changed files with 9 additions and 10 deletions

View File

@ -119,11 +119,11 @@ class _Visitor implements i18n.Visitor {
} }
visitPlaceholder(ph: i18n.Placeholder, context?: any): xml.Node[] { visitPlaceholder(ph: i18n.Placeholder, context?: any): xml.Node[] {
return [new xml.Tag(_PLACEHOLDER_TAG, {name: ph.name}, [], false)]; return [new xml.Tag(_PLACEHOLDER_TAG, {name: ph.name})];
} }
visitIcuPlaceholder(ph: i18n.IcuPlaceholder, context?: any): xml.Node[] { visitIcuPlaceholder(ph: i18n.IcuPlaceholder, context?: any): xml.Node[] {
return [new xml.Tag(_PLACEHOLDER_TAG, {name: ph.name}, [], false)]; return [new xml.Tag(_PLACEHOLDER_TAG, {name: ph.name})];
} }
serialize(nodes: i18n.Node[]): xml.Node[] { serialize(nodes: i18n.Node[]): xml.Node[] {

View File

@ -18,8 +18,7 @@ class _Visitor implements IVisitor {
const strAttrs = this._serializeAttributes(tag.attrs); const strAttrs = this._serializeAttributes(tag.attrs);
if (tag.children.length == 0) { if (tag.children.length == 0) {
return tag.canSelfClose ? `<${tag.name}${strAttrs}/>` : return `<${tag.name}${strAttrs}/>`;
`<${tag.name}${strAttrs}></${tag.name}>`;
} }
const strChildren = tag.children.map(node => node.visit(this)); const strChildren = tag.children.map(node => node.visit(this));
@ -72,8 +71,8 @@ export class Tag implements Node {
public attrs: {[k: string]: string} = {}; public attrs: {[k: string]: string} = {};
constructor( constructor(
public name: string, unescapedAttrs: {[k: string]: string} = {}, public children: Node[] = [], public name: string, unescapedAttrs: {[k: string]: string} = {},
public canSelfClose: boolean = true) { public children: Node[] = []) {
Object.keys(unescapedAttrs).forEach((k: string) => { Object.keys(unescapedAttrs).forEach((k: string) => {
this.attrs[k] = _escapeXml(unescapedAttrs[k]); this.attrs[k] = _escapeXml(unescapedAttrs[k]);
}); });

View File

@ -208,9 +208,9 @@ const XMB = `
<ph name="ICU"/> <ph name="ICU"/>
</msg> </msg>
<msg id="5868084092545682515">{VAR_SELECT, select, m {male} f {female} }</msg> <msg id="5868084092545682515">{VAR_SELECT, select, m {male} f {female} }</msg>
<msg id="4851788426695310455"><ph name="INTERPOLATION"></ph></msg> <msg id="4851788426695310455"><ph name="INTERPOLATION"/></msg>
<msg id="9013357158046221374">sex = <ph name="INTERPOLATION"></ph>></msg> <msg id="9013357158046221374">sex = <ph name="INTERPOLATION"/></msg>
<msg id="8324617391167353662"><ph name="CUSTOM_NAME"></ph></msg> <msg id="8324617391167353662"><ph name="CUSTOM_NAME"/></msg>
<msg id="7685649297917455806">in a translatable section</msg> <msg id="7685649297917455806">in a translatable section</msg>
<msg id="2387287228265107305"> <msg id="2387287228265107305">
<ph name="START_HEADING_LEVEL1"><ex>&lt;h1&gt;</ex></ph>Markers in html comments<ph name="CLOSE_HEADING_LEVEL1"><ex>&lt;/h1&gt;</ex></ph> <ph name="START_HEADING_LEVEL1"><ex>&lt;h1&gt;</ex></ph>Markers in html comments<ph name="CLOSE_HEADING_LEVEL1"><ex>&lt;/h1&gt;</ex></ph>

View File

@ -46,7 +46,7 @@ export function main(): void {
<!ELEMENT ex (#PCDATA)> <!ELEMENT ex (#PCDATA)>
]> ]>
<messagebundle> <messagebundle>
<msg id="7056919470098446707">translatable element <ph name="START_BOLD_TEXT"><ex>&lt;b&gt;</ex></ph>with placeholders<ph name="CLOSE_BOLD_TEXT"><ex>&lt;/b&gt;</ex></ph> <ph name="INTERPOLATION"></ph></msg> <msg id="7056919470098446707">translatable element <ph name="START_BOLD_TEXT"><ex>&lt;b&gt;</ex></ph>with placeholders<ph name="CLOSE_BOLD_TEXT"><ex>&lt;/b&gt;</ex></ph> <ph name="INTERPOLATION"/></msg>
<msg id="2981514368455622387">{VAR_PLURAL, plural, =0 {<ph name="START_PARAGRAPH"><ex>&lt;p&gt;</ex></ph>test<ph name="CLOSE_PARAGRAPH"><ex>&lt;/p&gt;</ex></ph>} }</msg> <msg id="2981514368455622387">{VAR_PLURAL, plural, =0 {<ph name="START_PARAGRAPH"><ex>&lt;p&gt;</ex></ph>test<ph name="CLOSE_PARAGRAPH"><ex>&lt;/p&gt;</ex></ph>} }</msg>
<msg id="7999024498831672133" desc="d" meaning="m">foo</msg> <msg id="7999024498831672133" desc="d" meaning="m">foo</msg>
<msg id="i" desc="d" meaning="m">foo</msg> <msg id="i" desc="d" meaning="m">foo</msg>