github-259: remove shiftFunction configuration

README.md

@@ -521,13 +521,6 @@ Default: `('passwd', 'password', 'secret', 'confirm_password', 'password_confirm
 <dd>Array of fields that you do NOT to be scrubbed even if they match entries in scrub_fields. Entries should be provided in associative array dot notation, i.e. `data.person.username`.
 </dd>
 
-<dt>shift_function
-</dt>
-<dd>Whether to shift function names in stack traces down one frame, so that the function name correctly reflects the context of each frame.
-
-Default: `true`
-</dd>
-
 <dt>timeout
 </dt>
 <dd>Request timeout for posting to rollbar, in seconds.

src/DataBuilder.php

@@ -45,7 +45,6 @@ class DataBuilder implements DataBuilderInterface
     protected $baseException;
     protected $includeCodeContext;
     protected $includeExcCodeContext;
-    protected $shiftFunction;
     protected $sendMessageTrace;
     protected $rawRequestBody;
     protected $localVarsDump;
@@ -99,11 +98,6 @@ public function __construct($config)
         $this->setLocalVarsDump($config);
         $this->setCaptureErrorStacktraces($config);
         $this->setLevelFactory($config);
-
-        $this->shiftFunction = $this->tryGet($config, 'shift_function');
-        if (!isset($this->shiftFunction)) {
-            $this->shiftFunction = true;
-        }
     }
 
     protected function tryGet($array, $key)
@@ -463,13 +457,6 @@ public function makeFrames($exception, $includeContext)
 
             $frames[] = $frame;
         }
-
-        if ($this->shiftFunction && count($frames) > 0) {
-            for ($i = count($frames) - 1; $i > 0; $i--) {
-                $frames[$i]->setMethod($frames[$i - 1]->getMethod());
-            }
-            $frames[0]->setMethod('<main>');
-        }
 
         $frames = array_reverse($frames);
 

tests/BackwardsCompatibilityConfigTest.php

@@ -50,7 +50,6 @@ public function testConfigValues()
             },
             'root' => '/Users/brian/www/app',
             'scrub_fields' => array('test'),
-            'shift_function' => false,
             'timeout' => 10,
             'report_suppressed' => true,
             'use_error_reporting' => true,