Hi Daniel and Joscha,

 

> using nested sets AND parent_id's makes the code hard to understand.
> You are mixing to different concepts which also doubles potential error
pits.

 

It’s a bit redundant, that’s true, I think of it like an index for fast
access. You have all the information in id/parentid/oxsort, so this can be
seen as “master”, and from there, nested sets structure is built. Oxid
itself uses nested sets so rarely that it might look like the overhead in
maintaining nested sets is not justified. But if you write a module and need
something like “get all childs and subchilds” it is nice to have nested sets
available.

 

Maybe nested sets could be dropped, I don’t know how much the performance
impact would really be if recursion was used in such cases, probably not
really notable. Talking about things that could be dropped, first two things
I would mention are Content Cats and Price Cats, which both seem close to
useless to me, and equally unnecessarily complicate the template and core
code in lots of places.

 

Best Regards, Frank

 

 

 

Von: [email protected]
[mailto:[email protected]] Im Auftrag von Joscha Krug
| marmalade.de
Gesendet: Dienstag, 17. Juli 2012 09:22
An: [email protected]
Betreff: Re: [oxid-dev-general] Nested Sets and OXSORT

 

Hi Daniel,

> using nested sets AND parent_id's makes the code hard to understand.
> You are mixing to different concepts which also doubles potential error
pits.

You've nailed it! Thats what i wanted to say. I can go with both concepts
alone. But mixing them is really strange! Although you could enter "OXSORT"
also for subcategories.

Best Joscha


//---------

 

marmalade.de
Joscha Krug

 

www.marmalade.de <http://www.marmalade.de/> 
[email protected]

 

Leibnizstr.25
39104 Magdeburg
GERMANY

 

phone: +49 (0) 391 / 559 22 104
fax:      +49 (0) 391 / 559 22 106

OXMOB | mobile Template
<http://www.oxmob.de/?pk_campaign=OXMOB%20%7C%20E-Mail-Link&pk_kwd=normalEma
il> 
Das einfach geniale OXID eShop Modul.
Ab sofort in unserem Online-Shop
<http://www.marmalade.de/shop/Templates/OXMOB-OXID-eShop-mobile-Template.htm
l?pk_campaign=OXMOB%20%7C%20E-Mail-Link&pk_kwd=normalEmail>  erhältlich.

_______________________________________________
dev-general mailing list
[email protected]
http://dir.gmane.org/gmane.comp.php.oxid.general

Reply via email to