[Natsheh]

Originally Posted by Bugsy So for every ArrayCreate() called, you output the handle and all are valid? Are you 110% certain that your handle storage mechanism is working correctly?

I didnt output the handle but i outputed the array counts but when an invalid array is returned from ArrayCreate i demolished the set_task from getting called.

Originally Posted by Bugsy I assume you changed these to 0? PHP Code: if(iTempArray == any:-1) //.. if(iTempArray != any:-1) ArrayDestroy(iTempArray);

No they've stayed as they were because -1 meaning the array is not created.

Originally Posted by Bugsy Have you tried passing the handle directly in set_task() and ensure you destroy each one when done?

I forget to mention after the error occurs within a while, bunch of plugins will start getting defecated meaning they will throw a bunch of random errors such as index out of bounds and other errors.

[Bugsy]

I thought it was determined that an invalid handle is 0, not -1?

If other plugins that also use the Array[X] natives also begin erroring when yours does then yeah I think it's a module issue.

Does the handle have a similar value range each time it begins erroring?

[Natsheh]

Originally Posted by Bugsy I thought it was determined that an invalid handle is 0, not -1? If other plugins that also use the Array[X] natives also begin erroring when yours does then yeah I think it's a module issue. Does the handle have a similar value range each time it begins erroring?

here's what i did at the temp array creating to prevent the error before calling the set_task.

Spoiler

PHP Code:

                        if(iTempArray == any:-1)
                        {
                            iTempArray = ArrayCreate(Hudmessage_SetTask_Data, 1);
                            
                            if(iTempArray == Invalid_Array)
                            {
                                server_print("Failed to create a temporary 'HUDMSG DATA PARAMS ARRAY'");
                                server_print("Task (%s) has failed to get on repeat!", xArray2[HUDMSG_ID]);
                                continue;
                            }
                        } 


Here're some random errors...

L 07/27/2021 - 11:45:18: Invalid player id 0
L 07/27/2021 - 11:45:18: [AMXX] Run time error 10 (plugin "cs_player_models_api.amxx") (native "set_user_info") - debug not enabled!
L 07/27/2021 - 11:45:18: [AMXX] To enable debug mode, add "debug" after the plugin name in plugins.ini (without quotes).
L 07/27/2021 - 11:45:18: Invalid array handle provided (0)


L 07/27/2021 - 11:45:18: [AMXX] Displaying debug trace (plugin "jb_effects_system.amxx", version "2.6.6")
L 07/27/2021 - 11:45:18: [AMXX] Run time error 10: native error (native "ArrayGetArray")
L 07/27/2021 - 11:45:18: [AMXX]    [0] jb_effects_system.sma::task_hudmessage_repeat (line 968)
L 07/27/2021 - 11:45:23: Invalid array handle provided (0)

L 07/27/2021 - 11:52:56: [AMXX] Displaying debug trace (plugin "Jailbreak_main.amxx", version "2.6.6")
L 07/27/2021 - 11:52:56: [AMXX] Run time error 10: native error (native "menu_item_getinfo")
L 07/27/2021 - 11:52:56: [AMXX]    [0] Jailbreak_main.sma::primary_wpnshandle (line 736)
L 07/27/2021 - 11:52:56: Invalid menu id 1(32)

i just found a bug or a potential bug if you may say, some of the players might not have any weapons although this still been called, so if it was called without any weapons in the menu, the menu will simply won't be displayed, so the menu is been created and its not destroyed and it might make a problem latter.

Spoiler

PHP Code:

Show_weaponsmenu(id, wpns[], const title[], const handler[])
{
    if(!Check_weapons_available(id, true) || wpns[0] == 0)
        return;
    
    trim(wpns);
    remove_quotes(wpns);
    
    new sOutput[MAX_WEAPONS][24], sInfo[24], szTitle[128];
    str_explode(wpns, ',', sOutput, MAX_WEAPONS, charsmax(sOutput[]));
    
    formatex(szTitle, charsmax(szTitle), "%L", id, title);
    new iMenu = menu_create(szTitle, handler);
    
    for( new i = 0; i < MAX_WEAPONS; i++)
    {
        strtolower(sOutput[i])
        formatex(sInfo, charsmax(sInfo), "weapon_%s", sOutput[i])
        
        if(!get_weaponid(sInfo))
            continue;
        
        menu_additem(iMenu, sOutput[i], sInfo);
    }

    menu_display(id, iMenu);
} 


PHP Code:

Show_weaponsmenu(id, wpns[], const title[], const handler[])
{
    if(!Check_weapons_available(id, true) || wpns[0] == 0)
        return;
    
    trim(wpns);
    remove_quotes(wpns);
    
    new sOutput[MAX_WEAPONS][24], sInfo[24], szTitle[128];
    str_explode(wpns, ',', sOutput, MAX_WEAPONS, charsmax(sOutput[]));
    
    formatex(szTitle, charsmax(szTitle), "%L", id, title);
    new iMenu = menu_create(szTitle, handler), iWeaponsCount;
    
    for( new i = 0; i < MAX_WEAPONS; i++)
    {
        strtolower(sOutput[i])
        formatex(sInfo, charsmax(sInfo), "weapon_%s", sOutput[i])
        
        if(!get_weaponid(sInfo))
            continue;
        
        menu_additem(iMenu, sOutput[i], sInfo);
        iWeaponsCount++;
    }

    if(!iWeaponsCount)
    {
        menu_destroy(iMenu);
        return;
    }


    menu_display(id, iMenu);
} 


[Bugsy]

When does iTempArray get set to -1?

[Natsheh]

Originally Posted by Bugsy When does iTempArray get set to -1?

PHP Code:

                    if((z=xArray2[HUDMSG_REPEAT]) >= 1) // repeats
                    {
                        new Array:iTempArray = any:-1, iTempArrayItem; 


[Natsheh]

After a while of observation the server didn't throw any error assuming the fix of the potential bug in post #23 has fixed the problem.

TIP always destroy the menus after done using them or if you didn't use them at all make sure they're destroyed.