| 00:00.41 | *** join/#arl Repo (n=supybot@repos.curseforge.net) |
| 00:00.41 | *** mode/#arl [+v Repo] by ChanServ |
| 01:36.40 | *** join/#arl ckknight (n=ckknight@WoWUIDev/WoWAce/CurseStaff/CurseForge/ckknight) |
| 04:10.34 | *** join/#arl ckknight (n=ckknight@WoWUIDev/WoWAce/CurseStaff/CurseForge/ckknight) |
| 04:20.18 | *** join/#arl ckknight (n=ckknight@WoWUIDev/WoWAce/CurseStaff/CurseForge/ckknight) |
| 08:43.50 | *** join/#arl EthanCentaurai (n=EthanCen@89.243.157.12) |
| 17:51.46 | *** join/#arl purl (i=ibot@rikers.org) |
| 17:51.46 | *** topic/#arl is RC9 released! http://tinyurl.com/4997z6 or http://tinyurl.com/a3o2mb | Database being updated and minor bugs being squashed for 1.0 release | ARL Stats! Claim you commits: http://tinyurl.com/kpnzk2 | WoWInsider review: http://bit.ly/HwKXO | http://ackis.myminicity.com/env |
| 18:18.42 | pompy | Ackis: when adding the skill levels, did you scroll back and forth?? cuz I cant like get the list sorted on wowhead the like same way it is in teh db lol. |
| 18:29.18 | Ackis | yah I ened up scrolling back and forth on pages =/ |
| 18:29.53 | pompy | that jus doesnt sound... fun |
| 18:30.14 | pompy | esp when on some u have to go thru ilke 400 recipes :x |
| 18:30.16 | Ackis | it's not that bad actually |
| 18:31.35 | pompy | u kno what i noticed too.. if you have 'unknown' checked.. and jus 'known' .. it includes the "ALL" total in teh count. |
| 18:32.07 | pompy | would prolly would want to only see the 'known' total of whatever filter you have checked, ya get whut im trying to say? |
| 18:33.20 | Ackis | no idea what you're trying to say ;) lol screen shot it for me please |
| 18:33.56 | Ackis | did torhal fix the pvp stuff? |
| 18:34.45 | pompy | no he didnt |
| 18:35.09 | Ackis | k I see the code is handled in the back end so it's just the frame that's having issues then |
| 18:35.22 | pompy | wheres it handled in the backend |
| 18:36.06 | pompy | like where arl shows #/# .. the number thats the total, includes the UNKOWN and KNOWN -total. Regardless if you have 1 of the 2 checked only. |
| 18:37.55 | pompy | Ackis: how long did it take u to add those bs skill levels that u just commited? lol |
| 18:38.17 | Ackis | 10 mins or so |
| 18:38.22 | pompy | jeez |
| 18:38.29 | Ackis | maybe a bit longer, slow PC here at work |
| 18:38.45 | pompy | most of its not copy and paste either right lol |
| 18:39.28 | Ackis | nope |
| 18:39.31 | Ackis | none of it |
| 18:39.42 | pompy | gah that woulda took me forever :x |
| 18:39.43 | pompy | lol |
| 18:40.04 | pompy | So, I just gave the goblin rocket fuel under alchemy the trainer flag.. |
| 18:40.14 | pompy | theres no really accurate flag to give it lo |
| 18:40.15 | pompy | l |
| 18:40.24 | Ackis | kk |
| 18:43.17 | pompy | but yeah there's only 29 recipes in the whole database that need the pvp flag |
| 18:43.22 | pompy | 27 from wg, 2 from halaa |
| 18:43.48 | Ackis | aye it was requested and found valuable by some people (myself included)... it should work now |
| 18:44.07 | pompy | i was trying to think of the elseif if the pvp flag is missing/extra |
| 18:44.21 | pompy | bc saying elseif(flags[9]) isnt enough |
| 18:44.47 | pompy | bc it jjus says the pvp flag is extra when u scan the wg qm |
| 18:45.36 | *** join/#arl EthanCentaurai (n=EthanCen@78.149.196.122) |
| 18:47.10 | Ackis | it should only say extra when you're at the vendor itself right? |
| 18:47.24 | pompy | yes |
| 18:47.33 | Ackis | remember the zone name for WG? |
| 18:47.38 | Ackis | where the QM is stationed? |
| 18:47.43 | pompy | yteah |
| 18:49.16 | Ackis | what is it? :P hehe |
| 18:49.40 | pompy | same as teh first check |
| 18:49.45 | pompy | wintergrasp fortress |
| 18:50.01 | Ackis | hmm not sure why it's popping up w/ the pvp flag being extra then |
| 18:50.14 | pompy | bc |
| 18:50.18 | pompy | all the check is doing is |
| 18:50.24 | pompy | elseif(flags[9]) |
| 18:50.30 | pompy | if it has the flag, say its extra |
| 18:51.16 | pompy | id say jus take the elseif out oll.onyl 29 recipes have teh pvp flag |
| 18:51.19 | pompy | and they all have it right now |
| 18:51.19 | pompy | lol |
| 18:52.23 | Ackis | the check is doing "are we a vendor", if yes then "are we in halaa or wintergrasp fortress and is the pvp flag set" then we need to set it, otherwise the pvp flag is extra |
| 18:53.05 | Ackis | ah I see |
| 18:53.28 | pompy | :) |
| 18:53.30 | pompy | yeah and the check to see if recipes have at least 1 obtain flag, we havent seen that output cuz its not in teh part for scanning the whole db, just for a vendor or maybe trainer too |
| 18:54.49 | Ackis | it is? |
| 18:55.06 | Ackis | line 1724 in the dataminer |
| 18:55.09 | pompy | yes considering |
| 18:55.09 | pompy | tinsert(t,"No obtain flag. " .. recipe_name .. " (" .. spellid .. ")") |
| 18:56.37 | Ackis | I'm still confused here |
| 18:56.50 | pompy | well |
| 18:56.53 | Ackis | that's line 1727 |
| 18:57.15 | pompy | nu uh thats 1724 like u said |
| 18:57.18 | pompy | well |
| 18:57.21 | pompy | my copy haha |
| 18:57.22 | pompy | but |
| 18:57.33 | pompy | what do we have to do to get that output? |
| 18:57.37 | pompy | Scan what |
| 18:57.59 | Ackis | eI added a couple comments in one of my pushes :P |
| 18:58.12 | Ackis | any scan will generate that output |
| 18:58.31 | pompy | ughh why the hell is the patch dlding so slow :(. 4% in 55 minutes? |
| 18:58.34 | Ackis | the only vendor specific stuff is around line 1570 |
| 18:58.55 | Ackis | and it ends around line 1585ish |
| 18:59.08 | Ackis | everything else is for all types of scans |
| 18:59.40 | pompy | odd, i never got a message about the goblin rocket fuel missing an obtain flag, in any of my scans |
| 19:00.00 | Ackis | did you ever do a full alchemy scan? |
| 19:00.06 | pompy | no |
| 19:00.19 | Ackis | that's probably why |
| 19:00.24 | pompy | lame |
| 19:00.31 | Ackis | goblin rocket fuel would never be on a vendor |
| 19:00.39 | pompy | i know |
| 19:00.51 | pompy | theres no accurate flag to put for it |
| 19:01.04 | pompy | it has to be a special case |
| 19:01.06 | Ackis | yah I agree with you there |
| 19:01.10 | pompy | same with inlaid mithril cylinder |
| 19:01.12 | Ackis | not worried about that at all |
| 19:01.30 | pompy | no, but i had to get it to show up in arl some how |
| 19:04.32 | Ackis | ok so pvp should be fixed, dataminer fixed, anything else bugging you pompy? :D |
| 19:05.53 | pompy | well the alert for 'you already know all recipes for this profession' is kind of annoying, zeksie brought it up the other day |
| 19:06.10 | pompy | sometimes the default text in teh search box doesnt clear when you click it |
| 19:06.11 | pompy | lol |
| 19:07.12 | Ackis | yah the search thing you mentioned I htink |
| 19:07.19 | Ackis | the alert == not going away :P |
| 19:07.28 | Ackis | there's options for you to hide it |
| 20:15.08 | *** join/#arl robo\afk (n=robotusc@50A2D989.flatrate.dk) |
| 20:15.55 | *** join/#arl robo\afk (n=robotusc@50A2D989.flatrate.dk) |
| 21:44.30 | *** join/#arl Torhal (n=callahan@74-132-207-20.dhcp.insightbb.com) |
| 21:44.30 | *** mode/#arl [+o Torhal] by ChanServ |