unistack-org/pofile
4
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #28 from dedesite/patch-1

Browse Source 
Avoid putting a space in empty comments
This commit is contained in:
Ruben Vermeersch 2017-03-20 10:26:46 +01:00 committed by GitHub
commit f035affef6
2 changed files with 12 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/po.js
View File

@ -20,12 +20,12 @@ PO.prototype.toString = function () {
if (this.comments) { if (this.comments) {
this.comments.forEach(function (comment) { this.comments.forEach(function (comment) {
lines.push('# ' + comment); lines.push(('# ' + comment).trim());
}); });
} }
if (this.extractedComments) { if (this.extractedComments) {
this.extractedComments.forEach(function (comment) { this.extractedComments.forEach(function (comment) {
lines.push('#. ' + comment); lines.push(('#. ' + comment).trim());
}); });
} }

12
test/write.js
View File

@ -2,12 +2,13 @@ var assert = require('assert');
var fs = require('fs'); var fs = require('fs');
var PO = require('..'); var PO = require('..');
function assertHasLine(str, line) { function assertHasLine(str, line, doNotTrim) {
var lines = str.split('\n'); var lines = str.split('\n');
var found = false; var found = false;
for (var i = 0; i < lines.length; i++) { for (var i = 0; i < lines.length; i++) {
if (lines[i].trim() === line) { var lineToCompare = doNotTrim ? lines[i] : lines[i].trim();
if (lineToCompare === line) {
found = true; found = true;
break; break;
} }
@ -53,6 +54,13 @@ describe('Write', function () {
assertHasLine(str, '#, fuzzy'); assertHasLine(str, '#, fuzzy');
}); });
it('write empty comment without an unecessary space', function () {
var input = fs.readFileSync(__dirname + '/fixtures/fuzzy.po', 'utf8');
var po = PO.parse(input);
var str = po.toString();
assertHasLine(str, '#', true);
});
it('write flags only when true', function () { it('write flags only when true', function () {
var input = fs.readFileSync(__dirname + '/fixtures/fuzzy.po', 'utf8'); var input = fs.readFileSync(__dirname + '/fixtures/fuzzy.po', 'utf8');
var po = PO.parse(input); var po = PO.parse(input);