Split the binary expressions if they use more than 80 chars

Author: chris-l

node_translators/bin.js

@@ -2,8 +2,16 @@
 'use strict';
 
 module.exports = function (node, indent) {
- var codegen = this.process.bind(this);
- return codegen(node.left, indent) +
- this.ws + node.type + this.ws +
- codegen(node.right, indent);
+ var codegen, str, firstpart, secondpart;
+ codegen = this.process.bind(this);
+
+ firstpart = codegen(node.left, indent);
+ secondpart = codegen(node.right, indent);
+ str = firstpart + this.ws + node.type + this.ws + secondpart;
+
+ if (str.length > 80) {
+ str = firstpart + this.ws + node.type + this.nl + indent + this.indent + secondpart;
+ }
+
+ return str;
 };

test/tests.spec.js

@@ -39,6 +39,12 @@ describe('Declare', function () {
  });
 });
 
+describe('BinaryExpressions', function () {
+ it('should use multiple lines if they are longer than 80 chars', function () {
+ expect(parseUnparse('<?php\n{echo "background:#F9F7D8;color:#A79F00; border:1px solid #F0EEBD;" . "background:#F9F7D8;color:#A79F00; border:1px solid #F0EEBD;" . "background:#F9F7D8;color:#A79F00; border:1px solid #F0EEBD;";}')).toBe([ '<?php', '', '{', ' echo "background:#F9F7D8;color:#A79F00; border:1px solid #F0EEBD;" .', ' "background:#F9F7D8;color:#A79F00; border:1px solid #F0EEBD;" .', ' "background:#F9F7D8;color:#A79F00; border:1px solid #F0EEBD;";', '}', '', '' ].join('\n'));
+ });
+});
+
 describe('acid1.php', function () {
  it('must correcty convert nested blocks', function (done) {
  fs.readFile('./test/spec/acid1-parsed.php', function (err, strparsed) {