Support Forums - XOOPS Web Application System

Subject:* <
Name/Email:* <
Message Icon:* <
Select* <
Message:* <	xx-Small x-Small Small Medium Large x-Large xx-Large Arial Courier Georgia Helvetica Impact Verdana Haettenschweiler Black Blue Brown Green Grey Orange Paper Purple Red White Yellow Click the Preview to see the content in action.
Options:* <	Enable Smiley Enable Xoops Code
Confirmation Code* <	5 - 3 = ? Input the result from the expression Maximum attempts you can try: 10
* <

MODULE_X is the helper class for module x.(module x can be the module itself) The above is without hard-code and very understandable for everybody. Then I like to discuss about your special need and your exampled code(which is off-topic so maybe it is better to be in another topic :D): Side note 1: I dont know these codes you wrote comes from what module? but they are not good. One can do this without JOIN by enhancing that [b]Easy way[/b] :D [quote] For example: displays list articles with category names. Easy way: 1 - read articles from database by using article class 2 - for every article object do access to database by using category class [/quote] why do you (or that module developer) think you cannot follow that easy way more effective and with much less codes than the JOIN way? It can be done just by [b]two queries.[/b] 1 - read articles from database by using article handler class [code] SELECT * FROM {$this->table} [/code] 2- for [b]all above article category Ids[/b] do access to database by using category handler class. [code] SELECT category_id, category_name FROM {$this->table} WHERE category_id IN {$articles_category_id} [/code] which $articles_category_id is all category ids you get from the first query. To implement the above you even dont need to write special function (function &getObjects($args)) :D All can be done in front side :D I commented the below code for you to see how it can be done with ease :D [code] $articleH = xoops_getmodulehandler('article', 'MODULE'); $categoryH = xoops_getmodulehandler('category', 'MODULE'); $criteriaArticle = new CriteriaCompo(); // continue your criteria here for $criteriaArticle // ... // START first query for articles // remember getAll($criteria = null, $fields = null, $asObject = true, $id_as_key = true) $articleObjs = $articleH->getAll($criteriaArticle); // END first query for articles $articles_category_id = array(); foreach ($articleObjs as $artObj) { $articles_category_id[] = $artObj->getVar("category_id"); // you can do any other thing here with articles!!! // ... } // START second query for category names $criteriaCat = new CriteriaCompo(); $criteriaCat->add(new Criteria("category_id", "(" . implode(", ", $articles_category_id) . ")", "IN"), "AND"); // you can add what fields you need here :D $catFields = array("category_id","category_name"); $catNames = $categoryH->getAll($criteriaCat, $catFields, false); // END second query for category names // FINISH!!! // usage: $catNames[$artObj->getVar("category_id")]["category_name"] [/code] As i said before the above code is: 1- most effective code to do the job (more than JOIN) 2 simple queries are better than one query with JOIN. 2- no need to write any code in the handler class :D 3- can be extend very easy. eg: if you need category_created time you can add the field. Side note 2: Anyway IMO your query will not work because you didnt add db prefix: instead of: [code] mymodule_article [/code] you have to use: [code] $this->db->prefix('mymodule_article') [/code] but using hadlers and helper class you will not need to think about such a annoying thing (like i must add prefix here or not) :D Side note 3: If you are very tight for using only one query you should add one field category_name to articles table. [code] //need add initVar('category_name') to Article class [/code] As the above comment after your query suggested, If you really need to display the category name alongside an article in [b]many links of your module[/b] it is better to add another field called "category_name" in mymodule_article table. But generally you dont need that and i dont advise it:D [/quote]" />