URL-Rewrite – Strip index.php from Magento Nginx GitHub Configuration

What is set in your app/etc/local.xml

It should contain something like

<admin>
    <routers>
        <adminhtml>
            <args>
                <frontName><![CDATA[admin]]></frontName>
            </args>
        </adminhtml>
    </routers>
</admin>

If frontName is different use that to login in your admin panel.

It turns out that the match(Zend_Controller_Request_Http $request) in the MageWorx_SeoSuite Router had a switch statement which resolved CMS, Rss, Category, Category_layer and other routes.

The problem was with the default (read: product) case of this switch statement. It was quite a unique situation (which explains why it only affected a handful of products).

It would;

• Check if the end of the url (i.e: the product urlKey) had a rewrite and if not;
• Check if the lowest level category of said product had a rewrite and if not;
• Load the product by ID and fetch the canonical Url from the MageWorx_SeoSuite helper class;
• Send a redirect to the canonical Url and update the response using an earlier defined HTTP response code depending on the configuration settings.
• exit();

If either of the first two conditional statement were false; the method would return false; (I.E: router did not match). However, if both of the first two conditionals evaluated to true and some other checks passed (canonical Url was not empty etc.); the statement would send the redirect and then simply exit();.

I'm new to the whole routing methology but I beleive there were a few things wrong with this;

1. The method did not return a boolean for all routes of the conditionals (E.G if a product and it's imediate category had no rewrite).
2. The method would "match" but was never dispatch'd.
3. This is based on my limited (undergraduate) knowledge but: this seems like really bad software design.

It's worth pointing out at this time, because I don't want to slate MageWorx without the facts: I am quite sure this code has been altered in the last 5 years (whereby which several third party companies could have been responsible).

In summary: I commented out the default: case statement in the switch so that the Magento core takes responsibility. The culprit Urls now go to a 404, no-route page, which is ideal as they are all old non-existent paths. I would not usually alter module code in this hap-hazard manner but I have documented this well and we are in a fortunate position that the new version of our website is under development, so the "fix" is temporary.