(NFC) CRM_Core_CodeGen_Util_Template - Remove PEAR formatting filter
authorTim Otten <totten@civicrm.org>
Thu, 21 Jul 2016 18:51:27 +0000 (11:51 -0700)
committerTim Otten <totten@civicrm.org>
Thu, 21 Jul 2016 18:53:17 +0000 (11:53 -0700)
This filter has several issues:
 * Fundamentally, we don't use PEAR style guide - we're closer to Drupal style guide.
 * Its only practical effect is to move squiggly braces... to the wrong place.
 * It tangentially pulls in PEAR dependencies.
 * It adds a ~4 seconds to GenCode (as measured on my laptop).

CRM/Core/CodeGen/Util/Template.php

index 2590301c347a96d78a74e78a7ed1914e18622dd8..b6387e48d4a52d14fb2c3c75d960421f2ec27501 100644 (file)
@@ -29,8 +29,6 @@ class CRM_Core_CodeGen_Util_Template {
       $this->beautifier = new PHP_Beautifier();
       $this->beautifier->addFilter('ArrayNested');
       // add one or more filters
-      $this->beautifier->addFilter('Pear');
-      // add one or more filters
       $this->beautifier->addFilter('NewLines', array('after' => 'class, public, require, comment'));
       $this->beautifier->setIndentChar(' ');
       $this->beautifier->setIndentNumber(2);