Quantcast
Viewing all articles
Browse latest Browse all 293689

layout.module uses anti-pattern for string translation

Note: There's no layout.module component, so choosing base system.

layout.module does t('@name @type'), where @type can be module and theme. It then does t('module') or t('theme') separately. It seems cleaner and more flexible in terms of translation to simply do t('@name module') and t('@name theme') conditionally.

Patch attached.

AttachmentSizeStatusTest resultOperations
layout-t.patch1.33 KBIdlePASSED: [[SimpleTest]]: [MySQL] 57,001 pass(es).View details | Re-test

Viewing all articles
Browse latest Browse all 293689

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>