Hello,

Here is the first beta of MadeBy v1.7.3 *YAY*

Changes in this beta:
Version 1.7.3 Beta 1
Thursday March 15th 2007 9pm by ds
[ADDED] Multilanguage support
[ADDED] A "Language Bar" -- Can change the Language MadeBy displays in per user. (Will store in a session)
[ADDED] The administator can choose what languages to support!
[CHANGED] The configuration button will be hidden unless logged in as Admin (Login at the Main Roster Configuration)
[ADDED] MadeBy is now compatable with WoWRosterDF!
[NEW] MadeBy need to run on WoW Roster v1.7.3+ (or WoW RosterDF v1.7.3+) to work correctly!


Version 1.7.3 Beta 2
Friday March 16th 2007 by ds
[FIXED]Links and Forms for DF functions


Version 1.7.3 Beta 3
Friday March 16th 2007 by ds
[FIXED] Anchor links for DF


I do not run WoWRosterDF so I really need someone to test this out for me. It should work but testing is always good!

Give me your thoughts on the multi language support. This is my first thought for this type of support so I could very well change before release, depends on your feedback.

Also coming up soon is a revamped Sorting. Right now it sucks and I know this.

Here are my tests on WoWRosterDF in DragonFly
LEO linking is on
http://www.drkalliance.com/wowrosterdf/ ... adeby.html

The install button doesn't point to the right url

Code: Select all

http://www.drkalliance.com/?roster_addon_name=madeby&action=install

Probably should be this

Code: Select all

http://www.drkalliance.com/wowrosterdf/addon/roster_addon_name=madeby/action=install.html

File: index.php
Line 88
Replace

Code: Select all

<a href="?roster_addon_name='.$_GET['roster_addon_name'].'&amp;action=install">Install</a>

With

Code: Select all

'.make_link('','Install',array('action'=>'install')).'

There are other places where this needs to be fixed as well
Since they are not used, it doesn't matter ATM
Line 93
Replace

Code: Select all

<a href="?roster_addon_name='.$_GET['roster_addon_name'].'&amp;action=upgrade">Update</a>

With

Code: Select all

'.make_link('','Upgrade',array('action'=>'upgrade')).'

Line 99
Replace

Code: Select all

<a href="?roster_addon_name='.$_GET['roster_addon_name'].'&amp;action=install">Reinstall</a>

With

Code: Select all

'.make_link('','Reinstall',array('action'=>'install')).'

---

Installation seems to work ok except the link to go to the config page

File: install.php
Line: 79
Replace

Code: Select all

"<a href='?roster_addon_name=madeby&action=config'>".$wordings[$roster_conf['roster_lang']]['MadeBy_Configure_txt']."</a>".

With

Code: Select all

make_link('',$wordings[$roster_conf['roster_lang']]['MadeBy_Configure_txt'],array('action'=>'config')).

---

Using the selection box also uses the wrong url

Code: Select all

http://www.drkalliance.com/addon.php?roster_addon_name=madeby&proffilter=Cooking&filterbox=

Code: Select all

<form action="addon.php"

Needs to be set correctly, I don't have any ideas for this one

---

All the character links are bad as well

Code: Select all

http://www.drkalliance.com/wowrosterdf/char.php/name=Zaryn/server=Doomhammer/action=recipes.html

The correct link would be

Code: Select all

http://www.drkalliance.com/wowrosterdf/char/cname=Zaryn/server=Doomhammer/action=recipes.html

I don't have any ideas for this either

What exactly does DF's getlink(); function do? Does it make the substitutions to make LEO links (ie, does it tack the .html at the end?). Maybe I should go and Install WoWRosterDF and check out the function. It would appear its more involved then I thought.

Thanks for the report.

Uploaded Beta 2 at the top of this thread. I fixed (I hope) the DF compatibility.




Version 1.7.3 Beta 2
Friday March 16th 2007 by ds
[FIXED]Links and Forms for DF functions

The sub-catagorys need to be fix. For Each Profession.

In RecipeList.php
Lines 200-2003

Code: Select all

      
foreach($recipe_types as $type)
{
   $content .=  '<a href="#'.$type.'">'.$type.'</a> - '."\n";
}


But Nice.

I love that the developers are coding with cms in mind.

Thanks for the report Jlaplaca. I fixed the issue (i hope) in Beta 3. It is posted at the top of this thread.

I moved all problems to the Debug Posting, Please Sticky It

Thanks
Jlaplaca