| 00:04.58 | Torhal | Ressy: self.AddTrainer |
| 00:05.02 | Torhal | Should be self:AddTrainer |
| 00:05.04 | Torhal | :D |
| 00:06.27 | Torhal | Use /script ARL:addon:DumpTrainers("profession_name") BTW |
| 00:06.39 | Torhal | er, ARL:DumpTrainers |
| 00:06.50 | Torhal | I need to write a datamine GUI, seriously. |
| 00:09.14 | Ressy | sorry. I wasn't dumping trainers because its usually faster for me to type itout. |
| 00:09.29 | pompy | Ressy: AGREED |
| 00:09.39 | Ressy | plus, theres comments in it, and I didn't wanna wipe them out. |
| 00:09.52 | pompy | Ressy: **hug** |
| 00:10.40 | Torhal | I love how people keep blaming this on ARL: http://o7.no/O6m97h |
| 00:10.46 | Torhal | Cross-server bug. |
| 00:11.28 | Torhal | I should say "We don't modify Blizzard stuff - we read from it" but it wouldn't matter because nobody reads. |
| 00:12.40 | Ressy | fixed |
| 00:12.43 | Ressy | pushed |
| 00:12.57 | Torhal | pompy: BTW - I apologize for being a douche and calling you out in public Friday night. |
| 00:13.15 | Ressy | kiddo's being cranky. I'll be afk, but I'll check msgs every bit. :) |
| 01:07.15 | Ackis | Torhal: I'm back (it's a bad day today), did you commit your changes? |
| 01:14.08 | Torhal | Ackis: I did. |
| 01:14.40 | Torhal | Ackis: The reason I got nothing back is that LPJ only iterates over things YOU know. |
| 01:14.46 | Torhal | Added more pets and BOOM. |
| 01:15.18 | Torhal | And the PetJournal can have multiple instances of the same pet. |
| 01:16.34 | Torhal | There's also an issue where you have to scan twice. |
| 01:18.39 | Ackis | that's somethign I can debug :D |
| 01:20.09 | Torhal | Also, GetSpellInfo() in AddCollectable() should never be used. |
| 01:20.15 | Ackis | agreed |
| 01:20.17 | Torhal | We pass NPC Ids |
| 01:21.28 | Torhal | All in all, the PetJournal makes me want to stab things. |
| 01:22.08 | Ackis | what does this mean: http://www.wowace.com/paste/s619ntvx5efydz3p/ ? |
| 01:23.48 | Torhal | Ackis: git pull origin 2.0 |
| 01:24.08 | Torhal | The stuff at the bottom tells you how to set up your config to do this automagically. |
| 01:56.25 | pompy | Ackis: |
| 01:56.26 | pompy | (9/9/2012 9:56:05 PM) Repo: collectinator: pompachomp master * 1.1.1-6-gf733229 / (3 files in 2 directories): [+2 commits] |
| 01:56.26 | pompy | (9/9/2012 9:56:05 PM) Repo: f733229: Added a new mob: Darkmoon Rabbit |
| 01:56.27 | pompy | (9/9/2012 9:56:06 PM) Repo: b7aba24: Added a new pet: Darkmoon Rabbit |
| 01:56.28 | pompy | wee |
| 01:57.55 | Ackis | danke |
| 02:15.51 | Ackis | ~wowitem 69452 |
| 02:15.51 | purl | http://www.wowhead.com/?item=69452 http://www.wowdb.com/item.aspx?id=69452 |
| 02:16.08 | Ackis | ~wowspell 69452 |
| 02:16.08 | purl | http://www.wowhead.com/?spell=69452 http://www.wowdb.com/spell.aspx?id=69452 |
| 02:18.47 | pompy | ah nice Ackis only knew of ~wowhead |
| 02:49.31 | pompy | Jagged Forest Emerald: <a href="http://www.wowhead.com/?spell=53996">53996</a> |
| 02:49.32 | pompy | <PROTECTED> |
| 02:49.32 | pompy | <PROTECTED> |
| 02:49.32 | pompy | 53996: Vendor flag was not set. |
| 02:50.42 | Torhal | No profession dump was done, then |
| 02:50.53 | Torhal | All I did was go in and remove the removed from game and retired. |
| 02:50.58 | Torhal | The scanner doesn't check for that. |
| 02:51.42 | pompy | um, i pushed a dump post vendor scan , and its still telling me that after the dump. |
| 02:56.32 | Torhal | Apparently we've never had a case where that's come up because the scanner doesn't check for a vendor before saying the flag is extra |
| 02:56.36 | Torhal | Heh. |
| 03:02.14 | pompy | its only supposed to tell us the flag was not set if this condition succeeds "if not recipe:HasFilter("common1", "VENDOR") and not recipe:HasFilter("common1", "SEASONAL") then" .... but it has the vendor flag |
| 03:03.31 | Torhal | Yes - it does than when you're scanning a vendor. When you're doing a general profession dump, it skips that part (because it doesn't know) |
| 03:04.47 | Torhal | It said in the output that it was missing the vendor, and I'm willing to bet that a dump wasn't done afterward - the vendor was manually added. |
| 03:05.43 | pompy | the dump added the vendor filter flag |
| 03:06.15 | Torhal | After going to the vendor physically? |
| 03:06.20 | Torhal | Or what were the circumstances? |
| 03:09.04 | pompy | (9/9/2012 10:40:38 PM) Repo: arl: pompachomp master * 2.4-59-g4830f2f Database (2 files in 2 directories): [+1 commit] Datamined the Jewelcrafting Supplies vendor: Anuur |
| 03:09.06 | pompy | so physical |
| 03:09.46 | Torhal | Yah - that's the only time it would be directly added. At that point ARL has no doubt that it NEEDS a vendor flag. |
| 03:09.55 | Torhal | But a general scan, no way of knowing. |
| 03:10.38 | pompy | at thjat point and every time afterwards apparantely.. ive /rl 'd 500 times and rescanned the vendor and same message |
| 03:11.48 | Torhal | Why have we never had that issue before? O.o |
| 03:11.50 | Torhal | looks. |
| 03:13.33 | pompy | RepVendor is interferring |
| 03:13.43 | pompy | i commented it out and its not showing that in the scan output |
| 03:14.52 | Torhal | Oh |
| 03:14.54 | Torhal | OH |
| 03:14.59 | Torhal | I'm really not here. |
| 03:15.16 | Torhal | F.REPUTATION is rep vendor. At that point, having F.VENDOR as well is counter-productive. |
| 03:15.29 | Torhal | So yes, it gets removed. |
| 03:17.35 | pompy | but in this case, it comes from a Rep Vendor and a regular vendor. |
| 03:17.42 | pompy | so both flags would be ok in this case. |
| 03:18.35 | Torhal | That's an edge case that didn't exist until today. I can remove the code that does that, since I actually only added it to transfer all of the rep guys from F.VENDOR to F.REPUTATION |
| 03:22.12 | Torhal | Ok, pushed. |
| 03:53.17 | Ackis | pompy: glyph of barkskin is still saying only glyph mastery, needs to be updated please |
| 04:09.32 | pompy | Ackis: what is it now? |
| 04:10.17 | Ackis | book of glyph mastery |
| 04:11.31 | pompy | Ackis: now as in the new acquire :P |
| 04:13.08 | pompy | wowhead will never ever ever be accurate with inscription!! lol ><>< |
| 04:13.09 | Ackis | that'd require me to update and reload my ui and stuff |
| 04:13.37 | pompy | Ackis: huh? You told me it needs to be updated... so updated to what? :P |
| 04:13.48 | Ackis | glyph mastery to northrend research |
| 04:13.53 | Ackis | sorry we were miscommunicating |
| 04:14.10 | pompy | though there was no confirmation that book mastery was completely removed. |
| 04:14.29 | pompy | so i think theres just an alternate way to get the book of glyph mastery ones |
| 04:14.53 | Ackis | yup |
| 04:28.11 | pompy | LF will be replaced by CRLF in Database/Recipes/Inscription.lua. |
| 04:28.11 | pompy | The file will have its original line endings in your working directory. |
| 04:28.15 | pompy | i forgot what that means |
| 04:28.19 | pompy | and if to worry |
| 04:35.45 | pompy | hm maybe |
| 04:36.10 | pompy | Ressy's editor did something, it was set to the UNIX format for EOL instead of "dos/windows". hm. |
| 04:38.46 | Ackis | she was running in unix maybe? :D |
| 04:41.40 | pompy | well so is torhal and never have this issue, who knows. |
| 08:30.19 | Torhal | Dafuq? |
| 12:22.33 | *** join/#arl Torhal (~torhal@c-71-207-224-241.hsd1.al.comcast.net) |
| 12:22.33 | *** mode/#arl [+o Torhal] by ChanServ |
| 13:11.55 | *** join/#arl Ressy (~resa1983@unaffiliated/ressy) |
| 15:04.00 | *** join/#arl Torhal (~torhal@c-71-207-224-241.hsd1.al.comcast.net) |
| 15:04.01 | *** mode/#arl [+o Torhal] by ChanServ |
| 17:03.05 | *** join/#arl Ressy (~resa1983@bas2-toronto61-2925080613.dsl.bell.ca) |
| 17:37.36 | *** join/#arl Ressy (~resa1983@unaffiliated/ressy) |
| 19:03.30 | pompy | Silly Russians.. http://s019.radikal.ru/i633/1209/0b/f42f7e4397c7.jpg .. i love how it's not even resizable!! .. and the npc names and shit showing "????????????????" .. lol maybe blizzard issue in some weird way |
| 23:00.15 | *** join/#arl Ressy (~Ressy@unaffiliated/ressy) |
| 23:40.21 | Ressy | anyone doing anything with arl? |
| 23:43.24 | pompy | yeah a lil. |
| 23:44.04 | Ressy | kk |
| 23:44.33 | Ressy | I'm curious with what they did with Abyssal Shatter |
| 23:44.45 | Ressy | all these trainers pop up saying they don't offer it anymore.. |
| 23:45.11 | Ressy | its still in the game... |
| 23:47.51 | Ressy | huh.. its on this trainer. |
| 23:51.10 | Ressy | 8 hr downtime tomorrow. 5.0.5 is tomorrow. |
| 23:57.20 | pompy | still with a continuation of old bugs i bet!! ><>< |
| 23:59.01 | Ressy | Glyph problem I thought was fixed in beta? |
