1. This site uses cookies. By continuing to use this site, you are agreeing to our use of cookies. Learn More.

Resolved Combo Staff returns 0 for quantity of Rune.@@@.getQuantity() for the non-combo runes it provides

Discussion in 'Developer Support' started by Snufalufugus, Dec 5, 2017.

  1. Snufalufugus

    Joined:
    Aug 23, 2015
    Messages:
    1,961
    Likes Received:
    757
    Code (Text):
    1. snufalufugusmuffin [2 hours ago]
    2. @party Rune.AIR.getQuantity() returns 0 with a `Dust battlestaff` equipped
    3.  
    4.  
    5. cloud [12 minutes ago]
    6. OSRS or RS3?
    7.  
    8.  
    9. cloud [12 minutes ago]
    10. Also, making a thread would ensure we'd see it
    11.  
    12.  
    13. snufalufugusmuffin [11 minutes ago]
    14. OSRS
    15.  
    16.  
    17. snufalufugusmuffin [11 minutes ago]
    18. i made a comment in my code for party to see too :lul:
    19.  
    20.  
    21. cloud [10 minutes ago]
    22. Comments may be overlooked unless we think something odd is up with the code around it
    23.  
    24.  
    25. snufalufugusmuffin [10 minutes ago]
    26. :thumbsup:
    27.  
    28.  
    29. cloud [8 minutes ago]
    30. Looking at the code though I would think it would work
    31.  
    32.  
    33. cloud [7 minutes ago]
    34. actually that may only apply to combination runes not combination staffs
    35.  
    36.  
    37. cloud [7 minutes ago]
    38. check other combination staffs and see what the results are
    39.  
    40.  
    41. snufalufugusmuffin [7 minutes ago]
    42. if its regex related i wont be much help
    43.  
    44.  
    45. cloud [6 minutes ago]
    46. and also see if using Rune.DUST.getQuantity() returns a proper value
    47.  
    48.  
    49. cloud [6 minutes ago]
    50. Party can handle the regex if you can debug the results
    51.  
    52.  
    53. snufalufugusmuffin [5 minutes ago]
    54. for mystic mud staff, earth and water both return 0
    55.  
    56.  
    57. snufalufugusmuffin [4 minutes ago]
    58. mud returns `2147483647`
    59.  
    60.  
    61. snufalufugusmuffin [3 minutes ago]
    62. so it looks like combo staffs are considered to be only combo runes, instead of combo runes AND the combo rune's individual runes
    63.  
    64.  
    65. cloud [3 minutes ago]
    66. that's appropriate then
    67.  
    68.  
    69. snufalufugusmuffin [3 minutes ago]
    70. ehhh idk
    71.  
    72. cloud [2 minutes ago]
    73. yeah that seems to be case based on the regex
    74.  
    75.  
    76. cloud [2 minutes ago]
    77. my second to last response was in regards to your comment about combo staf
    78.  
    79. cloud [1 minute ago]
    80. staffs only being considered for combo runes
    81.  
    82.  
    83. snufalufugusmuffin [1 minute ago]
    84. by appropriate do you mean that's the intended operation?  because if so, i strongly disagree
    85.  
    86.  
    87. cloud [< 1 minute ago]
    88. @party look into this when you get a chance and @snufalufugusmuffin please make a forum thread with your findings and appropriate tags
    --- Double Post Merged, Dec 5, 2017, Original Post Date: Dec 5, 2017 ---
    @Party
    --- Double Post Merged, Dec 5, 2017 ---
    scuffed

     
  2. Best Answer:
    Post #4 by Party, Dec 5, 2017
  3. Party

    Party Client Developer

    Joined:
    Oct 12, 2015
    Messages:
    3,708
    Likes Received:
    1,606
  4. Snufalufugus

    Joined:
    Aug 23, 2015
    Messages:
    1,961
    Likes Received:
    757
    1. cloud [7 minutes ago]
    2. check other combination staffs and see what the results are
    1. cloud [6 minutes ago]
    2. and also see if using Rune.DUST.getQuantity() returns a proper value
    1. snufalufugusmuffin [5 minutes ago]
    2. for mystic mud staff, earth and water both return 0


    3. snufalufugusmuffin [4 minutes ago]
    4. mud returns `2147483647`


    5. snufalufugusmuffin [3 minutes ago]
    6. so it looks like combo staffs are considered to be only combo runes, instead of combo runes AND the combo rune's individual runes
     
  5. Party

    Party Client Developer

    Joined:
    Oct 12, 2015
    Messages:
    3,708
    Likes Received:
    1,606
    Yeah I found the issue but the fix isn't working and it's a pain in the ass.
    --- Double Post Merged, Dec 5, 2017, Original Post Date: Dec 5, 2017 ---
    Fixed next release.
     
    Snufalufugus likes this.

Share This Page

Loading...