Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

/holo reload #3291

Closed
Sejeniclas opened this issue Aug 10, 2020 · 4 comments
Closed

/holo reload #3291

Sejeniclas opened this issue Aug 10, 2020 · 4 comments
Labels
invalid Not a Skript issue, already implemented, does not apply, etc.

Comments

@Sejeniclas
Copy link

Sejeniclas commented Aug 10, 2020

When i type /holo reload it makes this error
[15:53:19 ERROR]: [Skript] 1 log handler was not stopped properly! (at ch.njol.skript.expressions.ExprParse.get(ExprParse.java:169)) [if you're a server admin and you see this message please file a bug report at https://github.com/bensku/skript/issues if there is not already one
Its when i inserted this:

topmoney:
  location: world, -233.500, 68.000, 239.500
  lines:
  - '&8✦ &e&lOlympus &b&lSKYBLOCK &8✦'
  - '&7Top 10 &8(&bPenge&8)'
  - ''
  - '%leaderboards_money_top_1%'
  - '%leaderboards_money_top_2%'
  - '%leaderboards_money_top_3%'
  - '%leaderboards_money_top_4%'
  - '%leaderboards_money_top_5%'
  - '%leaderboards_money_top_6%'
  - '%leaderboards_money_top_7%'
  - '%leaderboards_money_top_8%'
  - '%leaderboards_money_top_9%'
  - '%leaderboards_money_top_10%'
  - '&8...'
  - '%leaderboards_money_own%'

The apis is from here:

function getLeaderboardMoney(p:player,type:string) :: string:
    loop {players::*}:
        "%loop-value%" parsed as offline player is set:
            set {_balance::%loop-value%} to bal of ("%loop-value%" parsed as offline player)
            replace all "$" with "fdfs" in {_balance::%loop-value%}
            send "%{_balance::%loop-value%}%" to {_p}
        else:
            remove loop-value from {players::*}
    loop {_balance::*}:
        add 1 to {_size}
        if {_low.to.high.list::%loop-value%} is not set:
            set {_low.to.high.list::%loop-value%} to loop-index
        else:
            set {_n} to 0
            loop {_size} times:
                set {_n} to {_n}+1
                {_low.to.high.list::%loop-value-1%.%{_n}%} is not set
                set {_low.to.high.list::%loop-value-1%.%{_n}%} to loop-index
                stop loop
    #wait 1 tick
    set {_n} to size of {_low.to.high.list::*}
    loop {_low.to.high.list::*}:
        set {_high.to.low.list::%{_n}%} to loop-value
        set {_n} to {_n}-1
    loop {_high.to.low.list::*}:
        add 1 to {_result}
        if {_result} = 1:
            set {_top1} to "&6##&e1 &8- &7%loop-value parsed as offline player% &8(&b%bal of loop-value parsed as offline player%&8)"
        else if {_result} = 2:
            set {_top2} to "&6##&e2 &8- &7%loop-value parsed as offline player% &8(&b%bal of loop-value parsed as offline player%&8)"
        else if {_result} = 3:
            set {_top3} to "&6##&e3 &8- &7%loop-value parsed as offline player% &8(&b%bal of loop-value parsed as offline player%&8)"
        else if {_result} = 4:
            set {_top4} to "&6##&e4 &8- &7%loop-value parsed as offline player% &8(&b%bal of loop-value parsed as offline player%&8)"
        else if {_result} = 5:
            set {_top5} to "&6##&e5 &8- &7%loop-value parsed as offline player% &8(&b%bal of loop-value parsed as offline player%&8)"
        else if {_result} = 6:
            set {_top6} to "&6##&e6 &8- &7%loop-value parsed as offline player% &8(&b%bal of loop-value parsed as offline player%&8)"
        else if {_result} = 7:
            set {_top7} to "&6##&e7 &8- &7%loop-value parsed as offline player% &8(&b%bal of loop-value parsed as offline player%&8)"
        else if {_result} = 8:
            set {_top8} to "&6##&e8 &8- &7%loop-value parsed as offline player% &8(&b%bal of loop-value parsed as offline player%&8)"
        else if {_result} = 9:
            set {_top9} to "&6##&e9 &8- &7%loop-value parsed as offline player% &8(&b%bal of loop-value parsed as offline player%&8)"
        else if {_result} = 10:
            set {_top10} to "&6##&e10 &8- &7%loop-value parsed as offline player% &8(&b%bal of loop-value parsed as offline player%&8)"
        if loop-value parsed as offline player is {_p}:
            set {_own} to "&6##&e%{_result}% &8- &7%loop-value parsed as offline player% &8(&b%bal of loop-value parsed as offline player%&8)"
    if {_type} = "1", "2", "3", "4", "5", "6", "7", "8", "9" or "10":
        if {_top%{_type}%} is set:
            return {_top%{_type}%}
        return "&6##&e%{_type}% &8- &7Ingen &8(&b...&8)"
    else if {_type} = "own":
        if {_own} is set:
            return {_own}
        return "&6##&e0 &8- &7%{_p}% &8(&b0&8)"
@TomYaMee
Copy link

If you have skript-mirror, you are running an outdated version. Update to v2 Ref: #3058

@FranKusmiruk FranKusmiruk added the waiting for reply The report needs a response from the reporter to determine course of action. label Aug 10, 2020
@bluelhf
Copy link
Contributor

bluelhf commented Aug 10, 2020

He's running skript-mirror 2.0.0-SNAPSHOT, it's probably another addon.

Update: He's running a custom and outdated version. Not much to do here.

@TPGamesNL
Copy link
Member

AFAIK this issue can occur with unfortunate timing with multiple threads that all are actively using (adding / removing) log handlers, such as an async call to ExprParse.

@FranKusmiruk FranKusmiruk added invalid Not a Skript issue, already implemented, does not apply, etc. and removed waiting for reply The report needs a response from the reporter to determine course of action. labels Aug 18, 2020
@FranKusmiruk
Copy link
Member

Closing as there was no response from the issuer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid Not a Skript issue, already implemented, does not apply, etc.
Projects
None yet
Development

No branches or pull requests

5 participants