biddyweb / substruct

Automatically exported from code.google.com/p/substruct
0 stars 0 forks source link

PATCH Some cart partials are not being used and some ajax helper methods makes reference to a DOM node that does not exist #101

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. It doesn't break anything, what doesn't means its right either.  

What version of the product are you using? On what operating system?
Trunk. Ubuntu 7.10

Please provide any additional information below.
Appears that the cart and cart_container partials arent used (if anyone
needs it I think they should hold it inside /app), the cart partial is
referenced in some places of store controller, but the actions can simply
render nothing returning an state of success (as the result of the partial
never goes to the screen anyway). In the views, the form_remote_tag or
link_to_remote helper methods can simply ommit the update option, then an
Ajax.Request object will be created instead of an Ajax.Updater. Anyway a
DOM node pointed by it is never manipulated, always an entire show_cart
view inside the modal window is shown or reloaded, using showPopWin() or
window.location.reload() on complete. 

Original issue reported on code.google.com by edmundo...@gmail.com on 28 Jun 2008 at 6:01

Attachments:

GoogleCodeExporter commented 9 years ago
I'm skeptical of getting rid of this in case people are relying on it when 
upgrading
from older versions.

Keeping this around for future consideration.

Original comment by subim...@gmail.com on 11 Aug 2008 at 10:47

GoogleCodeExporter commented 9 years ago
I think that if people want to continue using it they need to move it to /app 
its
outside the scope of the system maintain that. Methods can be deprecated, but 
about
views I don't know what is the best way to inform that it will be removed.

Original comment by edmundo...@gmail.com on 12 Aug 2008 at 7:39

GoogleCodeExporter commented 9 years ago
Just removing the files and telling people to move to /app doesn't work. If 
there's
not a comprehensive list of what will be going away people will be blindsided 
if they
upgrade, then have to go search out where the old files are.

I don't have a good solution for this now either, which is why the files are 
staying.

Original comment by subim...@gmail.com on 12 Aug 2008 at 8:02