WowAce

Hi!

First of all, great addon! Just loving it! :)

I'm using the alpha-version you posted in a comment bit further down and I encounter an error while playing. When I'm hovering with my pointer above a mob that's supposed to have an quest-item, nothing is shown.

I'm pretty sure its been there before but maybe I'm wrong...

As said before, great addon!

BR

First of all this is by far the best addon of its kind, no other comes even close to it so thank you Sorha2 for it!

Nobody else seems to have mentioned this one but I found a glitch which is if the addon is enabled and I open the map then all the quests that I have are shown and are expanded and if I want to collapse them by clicking on the sign [-] before them the quests collapse momentarily but then immediately expand again. If I disable the addon this problem is gone.

Another thing I realized is that before this crap pre-Legion patch when I entered an insta and opened the map only the quests I had for that particular insta showed up on the map's list and also I had a button to show all other quests. I'm not sure if that was a feature of this addon but I'm quite sure it was. Now that doesn't work either.

Not having the severe error message everyone else seems to be having, but since the patch, my Quest Progress Messages(the messages that appear in the middle of the screen showing stuff like 1/5 completed, 2/5, etc) have stopping showing. I can't seem to find anything in the addon that re-enables it, but I know for sure it is the addon since they start appearing again when I disable it. I love the addon and all, but without those messages it is unusable for me.

No worries, I am trying to learn the lua language and I was tired of errors. Does IsInProvenGround work as a replacement for IsInChallengeMode? I was trying to find good api references.

I cannot test it but I believe this will fix the errors.

https://1drv.ms/u/s!ABWSqRNunu7ng3Y

Seems to be working OK.  Will keep you updated.

Having this one error popup each time I log in:

Date: 2016-07-23 10:45:39
ID: 1
Error occured in: Global
Count: 2
Message: ..\AddOns\SorhaQuestLog\Modules\Scenarios.lua line 1320:
   attempt to call field 'IsChallengeMode' (a nil value)
Debug:
   SorhaQuestLog\Modules\Scenarios.lua:1320: UpdateMinion()
   SorhaQuestLog\Modules\Scenarios.lua:1919: MinionAnchorUpdate()
   SorhaQuestLog\Modules\Scenarios.lua:1281: CreateMinionLayout()
   SorhaQuestLog\Modules\Scenarios.lua:1897: MinionAnchorUpdate()
   SorhaQuestLog\Modules\Scenarios.lua:1057:
      SorhaQuestLog\Modules\Scenarios.lua:1049
   (tail call): ?
   [C]: ?
   [string "safecall Dispatcher[1]"]:9:
      [string "safecall Dispatcher[1]"]:5
   (tail call): ?
   ...face\AddOns\Atlas\Libs\AceAddon-3.0\AceAddon-3.0.lua:529: InitializeAddon()
   ...face\AddOns\Atlas\Libs\AceAddon-3.0\AceAddon-3.0.lua:644:
      ...face\AddOns\Atlas\Libs\AceAddon-3.0\AceAddon-3.0.lua:636
Locals:
None
AddOns:
  MoncaiCompare, v6.2.0
  Swatter, v5.21f.5579 (SanctimoniousSwamprat)
  ACP, v3.4.28
  AspectOff, v1.0
  Atlas, v1.33.00
  AtlasArena, v1.06.01
  AtlasBattlegrounds, v1.33.00
  AtlasBurningCrusade, v1.33.00
  AtlasCataclysm, v1.33.00
  AtlasClassicWoW, v1.33.00
  AtlasDungeonLocs, v1.33.01
  AtlasMistsofPandaria, v1.33.00
  AtlasOutdoorRaids, v1.33.00
  AtlasScenarios, v1.33.00
  AtlasTransportation, v1.33.00
  AtlasWarlordsofDraenor, v1.33.01
  AtlasWorldEvents, v3.15
  AtlasWrathoftheLichKing, v1.33.00
  AtlasMajorCitiesEnhanced, vv1.08
  BagBrother, v
  Bagnon, v7.0.3
  Bartender4, v4.7.1
  BlizzMove, vv1.4.1
  Capping, v7.0
  ColorPickerPlus, v
  DailyGlobalCheck, v
  DailyGlobalCheckLegionWorldQuests, v
  DBMCore, v
  DBMDefaultSkin, v
  DBMStatusBarTimers, v
  DoomShards, vv13
  DraenorTreasures, v6.2.4.21343
  DugisGuideViewerZ, v4.003
  fubaTanaanObjectives, v1.1
  GatherMate2, v1.36.1
  HGarri, v1.2
  HideObjectiveTracker, v$Revision: 19 $
  InFlightLoad, v6.2.001
  MandrillMount, v22.624
  Masque, v6.2.1
  MasqueDarkscail, v1
  MasterPlanA, vA
  MikScrollingBattleText, v5.7.144
  OmniCC, v7.0.1
  PallyPower, vv5.0.11
  Postal, vv3.5.8
  Prat30, v3.6.14      
  Prat30Libraries, v
  Quartz, v3.3.0
  Raven, vr347
  Recount, vv7.0.3b release
  Redprints, vv1.1.3
  Routes, vv1.5.3
  ScreenGrid, v2.0
  SexyMap, v2.4.29
  ShadowedUnitFrames, vv4.1.2
  SharedMedia, v3.0.6-187
  SharedMediaAdditionalFonts, v1.0.20
  SilverDragon, vv3.2.5
  SimpleChi, v1.3c
  SimpleComboPoints, v1.6a
  SimpleHolyPower, v1.6
  SimpleShadowOrbs, vfinal-release
  SimpleShamanResources, v1.1c
  SlideBar, v5.21f.5579 (SanctimoniousSwamprat)
  SorhaQuestLog, vv1.4.8.2
  BlizRuntimeLib_enUS v7.0.3.70000 <none>
  (ck=6b7)

If you make the edits below to Quest.lua and then disable Scenarios the errors should stop.

For those experiencing the error (pulled from Swatter):

Message: ..\AddOns\SorhaQuestLog\Modules\Quests.lua line 3558:
   attempt to index local 'msg' (a number value)

The fix is in the Quests.lua file, edit (new stuff in bold):

Lines 3387 - 3388:

    self:RawHookScript(UIErrorsFrame, "OnEvent", function(self, event, num, msg, ...)
        QuestTracker:HandleUIErrorsFrame(self, event, num, msg, ...)

Line 3555:

function QuestTracker:HandleUIErrorsFrame(frame, event, num, msg, ...)

Line 3566:

QuestTracker.hooks[frame].OnEvent(frame, event, num, msg, ...)

Thanks for this!

broken in prepatch :(  Thanks in advance for updating!

Hi Sorha, thanks for the awesome addon! Do you plan to keep it updated for Legion with all the new features as world quests etc?

Quote from Sorha2 >>

I plan to, though I am a little worried it might take me a day or two after launch to get things sensible again :(