7
Quote:
birdseed wrote:
Hi
Your arguments are good. Nevertheless, i have to realize that performance in XOOPS is a topic which is generally answered with "turn on smarty caching" ;)
That answer usually comes from advanced
users and what else do you expect? Many people use XOOPS who have no idea about classes, functions and SQL statements, so naturally this will be their "rule of everything"
Quote:
As I am requesting to join the core developer team, it might be MY job to optimize that.
Yes, an objective in XOOPS 2.4 will be refinement, performance - and new functionality, depending on the available resources.
However, you should keep in mind that readability of the code is very important in systems that are used as XOOPS is. We have to cater for the people, who want minor modifications and easy upgrade paths. We have to cater for them, even if they have little knowledge about PHP.
Granted, the "enable caching" is not always the good reply, but when speaking of performance, it is imperative to find the bottleneck. Is the bottleneck in the OO structure of XOOPS? Is it in the joins between well-indexed tables? I don't think so. Is it in long processings of unoptimised code? I think we can definitely improve there. Is it in too many and unoptimised SQL queries? Certainly a big area. Is it in the Smarty templating engine? Perhaps, but the benefits are considerable and the workarounds are - as Skalpa stated - present.
Pre-rendering e.g. the texts so that they don't have to go through the text sanitizer's preg_replace calls is something we are discussing and planning for XOOPS 4 - perhaps we can implement improvements in XOOPS 2.4, too. I know, I'd be very interested in hearing your ideas on this - as long as they give the same flexibility as the current code.
The core (and certainly many modules) can be improved, but I think there are better areas to focus on than the OO structure in general and the Criteria, Security and Object classes in particular.
Looking forward to discussing this with you as soon as XOOPS 2.2 stable is out.
"When you can flatten entire cities at a whim, a tendency towards quiet reflection and seeing-things-from-the-other-fellow's-point-of-view is seldom necessary."
Cusix Software