<table><tr><td style="">ilic accepted this revision.<br />ilic added a comment.<br />This revision is now accepted and ready to land.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D7523" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I'd say the patch is fine as it is. If this code is invoked through an intermediate library, that library's definition of TRANSLATION_DOMAIN will have no effect, instead kross' TRANSLATION_DOMAIN will be used. Therefore if applicationDomain is empty, then there is no place to fetch translation from, and untranslated string should be returned (as it will happen with this patch).</p>

<p>Alternatively, in case of empty applicationDomain some sort of warning could be produced.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R317 Kross</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7523" rel="noreferrer">https://phabricator.kde.org/D7523</a></div></div><br /><div><strong>To: </strong>aacid, ltoscano, ilic, aspotashev<br /><strong>Cc: </strong>Frameworks<br /></div>