mklooss / loewenstark_seo Goto Github PK

When a category is a child of an inactive category it should not be accessible.

For some reason Magento does not consider the character case when mapping a URL to a CMS page. So http://mydomain/imprint is the same as http://mydomain/Imprint is the same as http://mydomain/imPrint and so forth. Besides auto-generating a canonical tag with the page identifier as unique URL the user should be redirected to the very same URL. After all, some editor might enter the URL in a different spelling and then we would have duplicate content.

Some of the public methods in Observer.php should be refactored to a helper class for better reusage.

For instance cleanUrl and _setCanonicalHeader are two good candidates. If I ever need to set a canonical header in my own module I won't have to duplicate any code and can rely on a helper instead of using the public methods of the observer directly. Seems like a more natural approach.

Any objections?

I've set 'catalog/seo/product_use_categories' (in German 'Kategoriepfad für Artikel URLs verwenden') in the backend settings.

For some reason magento (using 1.9.1) still defaults to create canonical urls in product pages without the category as part of the url.

E.g.
shop.dev/product.html
instead of
shop.dev/category/product.html

The same applies for links visible in other places of the store (e.g. on the recently viewed sidebar).

Any hints? Probably not related to Loewenstark_Seo, but I tried this extension and hoped it might help/solve the canonical URLs.

Cheers,
Frederik

own Database Table.

store_id int 11 not null
category_ids varchar 50 not null
sku varchar varchar 150 nullable
request_path varchar 255 not null
redirect_to varchar 255 nullable
redirect_type int 3 nullable (301/302)

https://github.com/mklooss/Loewenstark_Seo/blob/master/app/code/community/Loewenstark/Seo/etc/system.xml#L142
https://github.com/mklooss/Loewenstark_Seo/blob/master/app/code/community/Loewenstark/Seo/Model/Observer.php#L50

using own backend_model instant of event

Like

/catalog/product/view/id/1
/catalog/category/view/id/1

find via Model or direct in the Database the URL (core_url_write)

Extended check for redirectIfDisabled

45b1083#commitcomment-6822249

http://www.michaelloy.de/magento-seo-paginierung-mit-relnext-relprev-und-noindexfollow/magento-seo-paginierung-mit-relnext-relprev-und-noindexfollow.html

Classname is camelcase

enable/disable option.
and workarounds for some extension.

Update documentation to better reflect the current status of the module.

Give usage examples of not so easy to grasp-features like phrases and robots.txt

if url is not the categorie url
set Canonical tag: yes/no (option)

Flat is enabled.

Fatal error: Call to undefined method Mage_Catalog_Model_Resource_Category_Flat_Collection::getId() in /home/[....]/public_html/.modman/Loewenstark_Seo/app/code/community/Loewenstark/Seo/Model/Observer.php on line 530

With the current way we have a double negative, and we don't want no education that.