Currently not working #36


  • Defect
Open
  • AngryGoatGames created this issue Jun 30, 2021

    2x BulkMail2\BulkMail.lua:296: attempt to perform arithmetic on upvalue 'numItems' (a nil value)
    [string "@BulkMail2\BulkMail.lua"]:296: in function <BulkMail2\BulkMail.lua:287>
    [string "@BulkMail2\BulkMail.lua"]:1362: in function `RefreshSendQueueGUI'
    [string "@BulkMail2\BulkMail.lua"]:413: in function <BulkMail2\BulkMail.lua:401>
    [string "@BulkMail2\BulkMail.lua"]:423: in function <BulkMail2\BulkMail.lua:421>
    [string "@BulkMail2\BulkMail.lua"]:811: in function `?'
    [string "@AdiBags\libs\AceHook-3.0\AceHook-3.0-9.lua"]:90: in function <...faceAdiBags\libs\AceHook-3.0\AceHook-3.0.lua:87>

    Locals:
    basePrice = 0
    (*temporary) = <function> defined @FrameXML\MoneyFrame.lua:292
    (*temporary) = "SendMailCostMoneyFrame"
    (*temporary) = nil

    (*temporary) = 8
    (*temporary) = nil
    (*temporary) = 8
    (*temporary) = "attempt to perform arithmetic on upvalue 'numItems' (a nil value)"
    sendCache = <table> {
    0 = <table> {
    }
    }
    next = <function> defined =[C]:-1
    GetSendMailItem = <function> defined =[C]:-1
    GetSendMailPrice = <function> defined =[C]:-1
    MoneyFrame_Update = <function> defined @FrameXML\MoneyFrame.lua:292
    numItems = nil

  • AngryGoatGames added a tag Defect Jun 30, 2021
  • generalscar88 posted a comment Jul 2, 2021

    This is a very easy fix (or seems to be). It might not be the proper fix, but I will leave that to the author when they have time to apply the correct fixes and update the mod.

     

    Line 98 on BulkMail.lua

     

    see that numItems is declared, but not as a number. The code doesnt seem to set that up (maybe it didnt need to in 9.05, I dont know, been years since I played with LUA - like Pre cata days).

     

    Make a new line (line 99). Add the following code.

     

    numItems = 0

     

    Line 99 Code example

     

    If game running, just reload ur UI and the mod starts working again. I didnt test everything, but it now lets me mass send without issues.

     

     


    Edited Jul 2, 2021

To post a comment, please login or register a new account.