search

dw250100785 / magento-w2p

Automatically exported from code.google.com/p/magento-w2p
0 stars 0 forks source link

Unused functions #129

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What's this stuff doing in the code ?
http://code.google.com/p/magento-w2p/source/browse/trunk/app/code/local/ZetaPrin
ts/Zpapi/Model/zp_api.php#210

Original issue reported on code.google.com by zetapri...@gmail.com on 12 Feb 2010 at 4:06

GoogleCodeExporter commented 9 years ago 
It results in an error message for at least one installation.

Original comment by zetapri...@gmail.com on 12 Feb 2010 at 4:07

GoogleCodeExporter commented 9 years ago 
Remove unused functions.

Original comment by zetapri...@gmail.com on 21 Feb 2010 at 6:16

GoogleCodeExporter commented 9 years ago 
1. make sure they are unused
2. provide a test plan to check that all remaining api functions are covered

Original comment by zetapri...@gmail.com on 6 Nov 2010 at 9:23

GoogleCodeExporter commented 9 years ago 
New functions has zetaprints prefix, old - zp
We've not using old functions for a long time.

Original comment by Anatoly....@gmail.com on 6 Nov 2010 at 9:38

GoogleCodeExporter commented 9 years ago 
Unused functions were removed. 

Diffs: http://code.google.com/p/magento-w2p/source/detail?r=1167

Original comment by Anatoly....@gmail.com on 8 Nov 2010 at 3:37

GoogleCodeExporter commented 9 years ago 
Not sure it's worth testing.
Include in the nearest release and we'll see if it falls over during other 
testing.

Original comment by zetapri...@gmail.com on 9 Nov 2010 at 4:40

GoogleCodeExporter commented 9 years ago

Original comment by Anatoly....@gmail.com on 9 Nov 2010 at 5:27

GoogleCodeExporter commented 9 years ago 
Is it in the trunk?
Released?

Original comment by zetapri...@gmail.com on 25 Nov 2010 at 6:29