Suggestions

Quick and easy way to see what recipes your character is missing

Moderators: zeryl, Teta68

Suggestions

Postby Teta68 » Wed Jun 25, 2008 5:42 pm

Suggest any new features you want in the Missing Recipes addon.

Regards,

Teta
I break for giant Cyber-Hamsters.
User avatar
Teta68
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 72
Joined: Fri Jan 19, 2007 12:15 pm
Location: Sweden

Suggestions

Postby Anaxent » Sun Jun 29, 2008 5:21 am

I would like to see this templated for roster. Right now there seems to be a display bug in the current version which when the addon is installed with in a ported application like dragonfly the page gets all fouled up.
http://www.altercationondunemaul.com/ro ... =c:27.html

edit: I will take a look at the code to see what needs to be updated to fix the formatting, but it still would be nice to start seeing all new addons templated
Last edited by Anaxent on Sun Jun 29, 2008 10:18 am, edited 1 time in total.
User avatar
Anaxent
WoWRoster.net Dev Team
WoWRoster.net Dev Team
 
Posts: 642
Joined: Tue Jul 04, 2006 6:27 am
Location: Phoenix, Az

Re: Suggestions

Postby Teta68 » Mon Jun 30, 2008 1:05 pm

Hi Anaxent,

You are absolutely right, I need to template the addon. It was originally created before the templates were implemented in Roster 2, that's why I never even thought of it. I'll take a look at it soon and get back to you when I have a working solution.

Regards,

Teta
I break for giant Cyber-Hamsters.
User avatar
Teta68
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 72
Joined: Fri Jan 19, 2007 12:15 pm
Location: Sweden

Re: Suggestions

Postby Teta68 » Thu Jul 17, 2008 3:08 pm

Hi Anaxent,

The latest version of roster has been templated. Please let me know if it is compatible with your DragonFly ported Roster.

Regards,

Teta
I break for giant Cyber-Hamsters.
User avatar
Teta68
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 72
Joined: Fri Jan 19, 2007 12:15 pm
Location: Sweden

Suggestions

Postby zanix » Thu Jul 17, 2008 10:40 pm

What might be nice is a progress bar

Show total number of available recipes compared to what the player has and display as a progress bar
Code: Select all
|------ 159/212 - 75% ------|            |
Read the Forum Rules, the WiKi, and Search before posting!
WoWRoster v2.1 - SigGen v0.3.3.523 - WoWRosterDF
User avatar
zanix
Admin
Admin
WoWRoster.net Dev Team
WoWRoster.net Dev Team
UA/UU Developer
UA/UU Developer
 
Posts: 5546
Joined: Mon Jul 03, 2006 8:29 am
Location: Idaho Falls, Idaho
Realm: Doomhammer (PvE) - US

Re: Suggestions

Postby Teta68 » Fri Jul 18, 2008 7:26 pm

zanix wrote:What might be nice is a progress bar

Show total number of available recipes compared to what the player has and display as a progress bar
Code: Select all
|------ 159/212 - 75% ------|            |

Added in 2.0.1.2. Look for it in the downloads section.

Just put it in, if you know a better place for it on the page, make the suggestion :)

Regards,

Teta
I break for giant Cyber-Hamsters.
User avatar
Teta68
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 72
Joined: Fri Jan 19, 2007 12:15 pm
Location: Sweden

Suggestions

Postby ScratchMonkey » Mon Aug 11, 2008 2:00 am

First, I absolutely love this addon. It's the thing I think I missed the most in going to 2.0.

I'd like to suggest filtering out recipes from rival "schools" of a profession that one will be unable to get. I tried it on my goblin engineer and all the gnome school recipes are showing up, leading to quite a bit of clutter of recipes I can't learn.

Engineering might also need a special case for the epic head pieces. These are granted based on the character class, so my hunter won't see or be able to get the head piece patterns for other classes.

Check it out here:

http://matureasskickers.net/wowroster/M ... es&a=c:136

Probably at least 75% of those recipes are ones she's ineligible for.
User avatar
ScratchMonkey
WR.net Expert
WR.net Expert
 
Posts: 212
Joined: Wed Jul 05, 2006 4:32 pm
Location: San Pablo, CA

Re: Suggestions

Postby Teta68 » Mon Aug 11, 2008 1:35 pm

I have the same problem with my engineer and my tailor. I don't have any other high level professions yet, but I guess it would go for almost every profession.
I haven't looked at it before, but I recal from a forum discussion in the 1.7 version of this addon that there was no sure way of identifying which subtree of a profession a character was following.
Also, there is no mention of class in the recipes for the epic headdress, so that may be a pain to find out as well.
I am going to look at this a bit closer, as I don't want the clutter myself ;) but I don't have the time for it at this time.

Regards,

Teta
I break for giant Cyber-Hamsters.
User avatar
Teta68
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 72
Joined: Fri Jan 19, 2007 12:15 pm
Location: Sweden

Suggestions

Postby ScratchMonkey » Tue Aug 12, 2008 2:54 am

I figure it will need some degree of hackery and won't be completely determined from the data. For the engineer and tailor specialization, it may be sufficient to see if the player has any recipe in one specialization, and suppress listing all in the other if found. If no recipes in either specialization are found on the player, list all of them. (Hence, someone who hadn't yet selected a specialization would see all recipes.)
User avatar
ScratchMonkey
WR.net Expert
WR.net Expert
 
Posts: 212
Joined: Wed Jul 05, 2006 4:32 pm
Location: San Pablo, CA

Suggestions

Postby zanix » Tue Aug 12, 2008 5:24 am

The spell book contains the specialization as far as I know
You search this db table for known specializations
Read the Forum Rules, the WiKi, and Search before posting!
WoWRoster v2.1 - SigGen v0.3.3.523 - WoWRosterDF
User avatar
zanix
Admin
Admin
WoWRoster.net Dev Team
WoWRoster.net Dev Team
UA/UU Developer
UA/UU Developer
 
Posts: 5546
Joined: Mon Jul 03, 2006 8:29 am
Location: Idaho Falls, Idaho
Realm: Doomhammer (PvE) - US

Re: Suggestions

Postby Wyren » Mon Aug 25, 2008 2:33 pm

Teta68 wrote:
zanix wrote:What might be nice is a progress bar

Show total number of available recipes compared to what the player has and display as a progress bar
Code: Select all
|------ 159/212 - 75% ------|            |

Added in 2.0.1.2. Look for it in the downloads section.

Just put it in, if you know a better place for it on the page, make the suggestion :)

Regards,

Teta
For some reason I don't get the progress bar showing...
http://roster.treehuggery.org/index.php ... ipes&a=c:9

A suggestion for future development would be to include the drop locations, like the "MadeBy" addon does for recipes.

Oh, and perhaps fix the formatting on the skill numbers.
Last edited by Wyren on Mon Aug 25, 2008 2:34 pm, edited 1 time in total.
Wyren
WR.net Journeyman
WR.net Journeyman
 
Posts: 81
Joined: Wed Jul 26, 2006 3:07 pm

Re: Suggestions

Postby Wyren » Mon Sep 15, 2008 2:10 pm

Wyren wrote:For some reason I don't get the progress bar showing...
http://roster.treehuggery.org/index.php ... ipes&a=c:9
Fixed it, for some reason it wouldn't add the config option for the progress bar to roster_addon_config, so I had to manually put it in using phpmyadmin.
Code: Select all
insert into `roster_addon_config` values(7,2020,'mr_show_progressbar',1,'radio{Show^1|Hide^0','mr_display_conf')
Wyren
WR.net Journeyman
WR.net Journeyman
 
Posts: 81
Joined: Wed Jul 26, 2006 3:07 pm

Re: Suggestions

Postby Teta68 » Mon Oct 27, 2008 9:42 pm

Moved this post by Holgi here, for obvious reasons.

Thanks for the update!
btw: the german users only see english tooltips.

The used Tooltip-Link is:
www.wowhead.com/?item=...ocale=deDE

but this one only leads to an english version if you try this link.
The DE-Link is
http://de.wowhead.com/?item=11151&locale=deDE

I changed the line below and then it works, but then fixed for DE:
Code: Select all
$plink = "http://www.wowhead.com/?item=".get_icon_link($parsedrecipes[$sindex][$dindex]['pat_link']);

into
Code: Select all
$plink = "http://de.wowhead.com/?item=".get_icon_link($parsedrecipes[$sindex][$dindex]['pat_link']);

Maybe this it the problem?
Holgi

I break for giant Cyber-Hamsters.
User avatar
Teta68
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 72
Joined: Fri Jan 19, 2007 12:15 pm
Location: Sweden

Re: Suggestions

Postby Teta68 » Mon Oct 27, 2008 9:53 pm

Teta68 wrote:Moved this post by Holgi here, for obvious reasons.

Thanks for the update!
btw: the german users only see english tooltips.

The used Tooltip-Link is:
www.wowhead.com/?item=...ocale=deDE

but this one only leads to an english version if you try this link.
The DE-Link is
http://de.wowhead.com/?item=11151&locale=deDE

I changed the line below and then it works, but then fixed for DE:
Code: Select all
$plink = "http://www.wowhead.com/?item=".get_icon_link($parsedrecipes[$sindex][$dindex]['pat_link']);

into
Code: Select all
$plink = "http://de.wowhead.com/?item=".get_icon_link($parsedrecipes[$sindex][$dindex]['pat_link']);

Maybe this it the problem?
Holgi

I'm trying this:
Code: Select all
switch ($roster->data['clientLocale']) {
 case 'enUS':
  $plink = "http://www.wowhead.com/?item=".get_icon_link($parsedrecipes[$sindex][$dindex]['pat_link']);
  break;
 case 'deDE':
  $plink = "http://de.wowhead.com/?item=".get_icon_link($parsedrecipes[$sindex][$dindex]['pat_link']);
  break;
 case 'frFR':
  $plink = "http://fr.wowhead.com/?item=".get_icon_link($parsedrecipes[$sindex][$dindex]['pat_link']);
  break;
 case 'esES':
  $plink = "http://es.wowhead.com/?item=".get_icon_link($parsedrecipes[$sindex][$dindex]['pat_link']);
  break;
/* case 'ruRU':
 $plink = "http://ru.wowhead.com/?item=".get_icon_link($parsedrecipes[$sindex][$dindex]['pat_link']);
 break;
*/            }

Could you try it on your german site and see if it works? If it will I'll update the addon.
I also commented out the russian option as I am not sure if the locale is ruRU and right now I can't be bothered to find out and I don't have any other Russian localization for my addon anyway.

Regards,

Teta
Last edited by Teta68 on Mon Oct 27, 2008 10:05 pm, edited 3 times in total.
I break for giant Cyber-Hamsters.
User avatar
Teta68
Roster AddOn Dev
Roster AddOn Dev
 
Posts: 72
Joined: Fri Jan 19, 2007 12:15 pm
Location: Sweden

Suggestions

Postby ScratchMonkey » Mon Oct 27, 2008 10:49 pm

Am I correct that the only thing changing in each case above is the prefix to the wowhead.com domain? If so, assign that prefix in the cases to a temporary variable (say, $wowhead_subdomain), and use only one instance of the assignment statement. That's a lot easier to maintain, as only the changing part is in the switch.

Example:

Code: Select all
 case 'enUS':
  $wowhead_subdomain = "www";
  break;


Code: Select all
  $plink = "http://".$wowhead_subdomain."wowhead.com/?item=".get_icon_link($parsedrecipes[$sindex][$dindex]['pat_link']);


Then you can move all the subdomains to a localization file, making it easy to add new languages.
Last edited by ScratchMonkey on Mon Oct 27, 2008 10:50 pm, edited 1 time in total.
User avatar
ScratchMonkey
WR.net Expert
WR.net Expert
 
Posts: 212
Joined: Wed Jul 05, 2006 4:32 pm
Location: San Pablo, CA

Next

Return to Missing Recipes

Who is online

Users browsing this forum: No registered users and 1 guest

cron