Bug in admin/controller/extension/modification.php

[mhcwebdesign]

The following OCMOD xml results in an unexpected 'Not Found' error for OpenCart 2.0.1.1:

.....
<file path="catalog/view/theme/*/template/product/product.tpl">
        <operation>
            <search index="1"><![CDATA[<?php foreach ($options as $option) { ?>]]></search>
            <add position="replace"><![CDATA[blah blah<?php foreach ($options as $option) { ?>]]></add>
        </operation>
    </file>
.....

The corresponding VQMOD xml works as expected:

....
    <file name="catalog/view/theme/*/template/product/product.tpl">
        <operation>
            <search position="replace" index="1"><![CDATA[<?php foreach ($options as $option) { ?>]]></search>
            <add><![CDATA[blah blah<?php foreach ($options as $option) { ?>]]></add>
        </operation>
    </file>
....

Suggested bugfix:
The reason it fails in OCMOD is because of the index-attribute!
The index attribute should be 1-based, as in VQMOD, not zero-based.
In admin/controller/extension/modification.php, the lines around

                                        // Get all the matches
                                        $i = 0;

should be replaced with

                                        // Get all the matches
                                        $i = 1;