In RecipeList.php
Line 108

This Line of code is not working correcty on a postback.
The Problem is that the ? after addon should be &. This could be fixed
in your madeby_common.php Under Make_Form Function.

This is what it trys to do

Code: Select all

http://twistedjustice.net/index.php?name=WoWRosterDF&file=addon?roster_addon_name=madeby

Code: Select all

$choiceForm = make_form('addon.php?roster_addon_name=madeby', 'myform', 'POST');


In RecipeList.php
Line 200

The sub-category list is still not working for a Dragonfly System.

This is what it trys to do

Code: Select all

http://twistedjustice.net/index.php?name=#Consumable

Version 1.7.3 Beta 4
Saturday March 17th 2007 by ds
[FIXED]Fixed more errors in DF URL/Form handling.

See if this fixes your errors.

line 438 RecipeList.php

Code: Select all

$content .= '<br /><br /><br />'.border('sblue','start').make_link('',"  {$wordings[$lang]['MadeBy_Configure_txt']}  ", array('action'=>'install'))


needs to be

Code: Select all

$content .= '<br /><br /><br />'.border('sblue','start').make_link('',"  {$wordings[$lang]['MadeBy_Configure_txt']}  ", array('action'=>'config'))


What? You don't like a reinstall every time you want to configure an option?! Sheesh Fine.. I'll fix it.

Thanks for the catch...

Damn it.. just noticed some leftover debug crap in Beta 4.

ds001 wrote:What? You don't like a reinstall every time you want to configure an option?! Sheesh Fine.. I'll fix it.

Thanks for the catch...

lol well I do like the fact it lets me know it is installed correctly though....

Good, the link for the sub-category's almost working now.

The bookmarks that HTML works in Dragonfly, but when you select a profession and then click on the bookmark it only postback the bookmark, and not the currently selected profession.

Ex:

Code: Select all

http://twistedjustice.net/index.php?name=WoWRosterDF&file=addon&roster_addon_name=madeby#Consumable

Then Works if the Profession

Code: Select all

http://twistedjustice.net/index.php?name=WoWRosterDF&file=addon&roster_addon_name=madeby&proffilter=Alchemy#Consumable

It would also be nice to be able to hide the Item Description: as well due to having tooltips for the item icon.

And inverse sort level .. high end items before the crap lvl 1-40 items.

When I'm trying to reinstall the beta version of madeby I obtained that:

Code: Select all

Fatal error: mysql_free_result(): supplied argument is not a valid MySQL result resource in /mnt/116/sdb/3/8/sdroster2/roster/lib/wowdb.php on line 269

I have dropped the table in the database. So when I click on "madeby", I have the Installation windows. But as soon as I click on "install", I obtained the error messages.

I would assume your using Mysqli layer too eh?

I fixed that problem with the next beta. It should be released tonight after I fix one more annyoing DF issue.

Then after that I can actually start on new features!

cooooooool thanks DS

I should have updated before madeby. But WoW life takes times (aaaaand this b*&*&* of Aran)

i have one question concerning the "Level" column. Which level should be shown there?? Is it the needed crafting level e.g 315 or is it the chraracterlevel which is needed to equip that item??

The last option would be more useful i think. But the column is completely empty. So i wonder whats the use of it?

SethDeBlade wrote:i have one question concerning the "Level" column. Which level should be shown there?? Is it the needed crafting level e.g 315 or is it the chraracterlevel which is needed to equip that item??

The last option would be more useful i think. But the column is completely empty. So i wonder whats the use of it?

I agree 100%. Currently the Level column is the Character Level required to use/equip the item. I think i'll make another column (that can be toggled) called Skill Level that will show the required 'Trade Skill Level' to create the item.

I also need to make the sorting of the columns work better. Right now it is a mess.

Any thoughts on the Multi-Language option? Is it working? How could It be better?

ok. But the level column is empty in my rosters. Is this intended or a bug??

one suggestion. If selecting one tradeskill in a language e.g. enUS a link would be helpfull which says an top of the recipes "There are X guildmemmers available which have that skill to" and this link could leed to another language version e.g. deDE