aimeos / ai-typo3

TYPO3 adapter for Aimeos web shops and e-commerce solutions
https://aimeos.org/TYPO3
GNU Lesser General Public License v3.0
72 stars 6 forks source link

Always use plugin parameter #3

Closed gilbertsoft closed 7 years ago

gilbertsoft commented 7 years ago

To support the eID dispatcher

coveralls commented 7 years ago

Coverage Status

Coverage decreased (-6.4%) to 81.071% when pulling c570e46933f6c41231b96331c0b89eedc3a586a1 on GsAimeos:master into 0197533a506a64d3d23307c1ee2c84e12d872ea9 on aimeos:master.

coveralls commented 7 years ago

Coverage Status

Coverage decreased (-0.3%) to 87.158% when pulling c570e46933f6c41231b96331c0b89eedc3a586a1 on gilbertsoft:master into 0197533a506a64d3d23307c1ee2c84e12d872ea9 on aimeos:master.

aimeos commented 7 years ago

Is the "plugin" parameter something that needs to be namespaced, e.g. ai[plugin]?

gilbertsoft commented 7 years ago

Don't think so. All parameters were already implemented, but various parameters were only included under some special circumstances. No parameter is used by the plugin, this are informations for typo3 only to call the right plugin. Here are some example requests:

80.93.42.133 - - [04/Feb/2017:00:45:33 +0100] "GET /produkte/?eID=aimeos&plugin=catalog-count HTTP/1.1" 200 1576 "http://www.zwergenfabrik.ch/produkte/" "Mozilla/5.0 (Windows NT 10.0; WOW64; rv:51.0) Gecko/20100101 Firefox/51.0"

80.93.42.133 - - [04/Feb/2017:00:50:36 +0100] "GET /?ai%5Bs_prodcode%5D%5B0%5D=2510000000002&eID=aimeos&plugin=catalog-stock HTTP/1.1" 200 1285 "http://www.zwergenfabrik.ch/detail/d/281/Body/0/" "Mozilla/5.0 (Windows NT 10.0; WOW64; rv:51.0) Gecko/20100101 Firefox/51.0"

80.93.42.133 - - [04/Feb/2017:01:15:30 +0100] "GET /produkte/?eID=aimeos&plugin=catalog-suggest&ai%5Bf_search%5D=bei HTTP/1.1" 200 549 "http://www.zwergenfabrik.ch/produkte/" "Mozilla/5.0 (Windows NT 10.0; WOW64; rv:51.0) Gecko/20100101 Firefox/51.0"

gilbertsoft commented 7 years ago

Or do you ask because the if block with the namespacing?

aimeos commented 7 years ago

Yes, that was the reason

aimeos commented 7 years ago

Could you please add a unit test that covers the new plugin line so the tests are green?

gilbertsoft commented 7 years ago

I'm working on an new solution for issue #21 of aimeos-typo3