Hi there,
looking into model file product.php in frontend part od redShop remindes Virtuemart to me. Mix of PHP, HTML and inline styles in spaggetti form.
Tons of IFs, FOREACHs that could be puted in separate methods, maybe somewhere in helper. But this is your logic, I'm not going to edit that ;-)
But what I want to have possiblity to modify is template... But how with this in model?
| Code: |
<div style="float:left;width:500px;margin-left:-10px;border:0px solid #ff0000;">
|
or
| Code: |
<div style='margin:0px;padding:7px;' class='accessory_image
|
or other milions of line with inline styles and code! You can't be serious about this. Let's roll back into virtuemart with new component written from scratch?
Even if there's no inline style, there should not be even pure semantic html code, it's a MODEL guys! Application logic, not view....
Hope this is just some "Alpha state" of code and it's there just becouse you are busy with another work and it's in tracker for next releases already.... becouse if not, I'm considering to try a luck in another house, real MVC house (this post is a little bit related to one I've posted before - moving templates to files)....
Or did I miss something?