[PinHeaDi]

PHP Code:

public void OnMapStart()
{
    checkmap();
} 


PHP Code:

void checkmap()
{
    if(!GetConVarBool(RespawnEnabled))
        return;

    char line[128], g_szConfig[PLATFORM_MAX_PATH], g_szMap[128];
    
    GetCurrentMap(g_szMap, sizeof(g_szMap));
    BuildPath(Path_SM, g_szConfig, sizeof(g_szConfig), "configs/ar_respawn_maps.cfg.cfg");
    
    Handle MgMaps = OpenFile(g_szConfig, "r");
    
    while(!IsEndOfFile(MgMaps))
    {
        ReadFileLine(MgMaps, line, sizeof(line));
        
        if (StrEqual(line, g_szMap, false))
        { 
            g_bEnableMap = true;
            LateForRespawn = false;
        }
        CloseHandle(MgMaps);
    } 
    else 
    {
        PrintToServer("[SM] Could not find config (configs/ar_respawn_maps.cfg)");
    }
} 


It only reads the last line of the config, so only the last map in the list, not all of them.

ar_respawn_maps.cfg.cfg:

mg_acrophobia_run_v1
mg_adev
mg_antique_course
mg_cartoon_course_v3_1
mg_course_ravine_final
mg_crash_bandicoot_course_v1
mg_cyber_course_v5
mg_damons_course_beta
mg_dans_course_remaster
mg_dev_galaxie_course_csgo
mg_dev_netrial_course
mg_dust_course_2_csgo_v1
mg_escape_castle_v3_fix
mg_gavle_course_v5
mg_lego_course_3_csgo
mg_metal_course_2
mg_metal_course_level
mg_metal_course_level_2
mg_minecraft_adventure_final_fix
mg_minecraft_course_JB_N1
mg_nimafa_3_final_v3
mg_nxr_course_csgo
mg_office_coursee_2012_fixed
mg_runordie_final
mg_savethebacon_course_final
mg_savetheisland_course_reskin
mg_saw_3_v1
mg_siberdev_2
mg_sky_realm_v3
mg_sonic_islands_v4
mg_switch_cource
mg_last_light_v3
mg_touchdev_course_fix
mg_tunnel_beta_cource
mg_office45_course
mg_pharaohs_tomb2_go_v3
mg_onetaprun_course_v2_d
mg_ancient_course_v2
mg_blood_sector_v1
mg_east_office_beta
mg_ghosts_cave_csgo
mg_halo_course_csgo
mg_heppy_dev_v1
mg_office_easy_course_csgo

[Mitchell]

first thing i see is that you close the handle to the file in the while loop..
second thing i see is: configs/ar_respawn_maps.cfg.cfg

[PinHeaDi]

PHP Code:

void checkmap()
{
    if(!GetConVarBool(RespawnEnabled))
        return;

    char line[128], g_szConfig[PLATFORM_MAX_PATH], g_CurrentMap[128];
    
    GetCurrentMap(g_CurrentMap, sizeof(g_CurrentMap));
    BuildPath(Path_SM, g_szConfig, sizeof(g_szConfig), "configs/ar_respawn_maps.cfg");
    
    Handle MgMaps = OpenFile(g_szConfig, "r");
    
    if (MgMaps != INVALID_HANDLE)
    {
        while(!IsEndOfFile(MgMaps))
        {
            ReadFileLine(MgMaps, line, sizeof(line));
            if (StrEqual(line, g_CurrentMap, false))
            { 
                g_bEnableMap = true;
                LateForRespawn = false;
            { 
            else
            {
                g_bEnableMap = false;
            }
        }
        CloseHandle(MgMaps);
    }
    else
    {
        PrintToServer("[SM] Could not find config (configs/ar_respawn_maps.cfg)"); 
    } 


Still, it doesn't load as it should, only the last line /map/ in the config.

[Mitchell]

you should loop ReadFileLine instead of IsEndOfFile
https://github.com/MitchDizzle/Would...Rather.sp#L252

[PinHeaDi]

So, since I can't test the code right now. Should it be something like that:

PHP Code:

void checkmap()
{
    if(!GetConVarBool(RespawnEnabled))
        return;

    char line[128], g_szConfig[PLATFORM_MAX_PATH], g_CurrentMap[128];
    
    GetCurrentMap(g_CurrentMap, sizeof(g_CurrentMap));
    BuildPath(Path_SM, g_szConfig, sizeof(g_szConfig), "configs/ar_respawn_maps.cfg"); 
    
    File MgMaps;
    MgMaps = OpenFile(g_szConfig, "r");
    
    if (MgMaps != INVALID_HANDLE)
    {
        while(MgMaps.ReadLine(line, sizeof(line)))
        {
            if (StrEqual(line, g_CurrentMap, false))
            { 
                g_bEnableMap = true;
                LateForRespawn = false;
            }
            else{
                g_bEnableMap = false;
            }
        }
        MgMaps.Close();
    }
    else{
        PrintToServer("[SM] Could not find config (configs/ar_respawn_maps.cfg)"); 
    }
} 


It compiles without errors.

[Mitchell]

you would need something like this, or g_bEnableMap will never be true unless you were on the very last map in the list.

void checkmap() 
{ 
    if(!GetConVarBool(RespawnEnabled)) 
        return; 

    char line[128], g_szConfig[PLATFORM_MAX_PATH], g_CurrentMap[128]; 
     
    GetCurrentMap(g_CurrentMap, sizeof(g_CurrentMap)); 
    BuildPath(Path_SM, g_szConfig, sizeof(g_szConfig), "configs/ar_respawn_maps.cfg");  
     
    File MgMaps; 
    MgMaps = OpenFile(g_szConfig, "r"); 
     
    g_bEnableMap = false; 
    if (MgMaps != null) 
    { 
        while(MgMaps.ReadLine(line, sizeof(line))) 
        { 
            if (StrEqual(line, g_CurrentMap, false)) 
            {  
                g_bEnableMap = true; 
                LateForRespawn = false;
                break;
            }
        } 
        MgMaps.Close(); 
    } 
    else{ 
        PrintToServer("[SM] Could not find config (configs/ar_respawn_maps.cfg)");  
    } 
}

[PinHeaDi]

Will try that later, but I have a bool g_bEnableMap = false; at the beginning of the plugin or get that wrong. Isn't that bool supposed to put g_bEnableMap false by default.

P.S.: Nvm, it wasn't set to false, only bool g_bEnablemap;.

[Mitchell]

Originally Posted by PinHeaDi Will try that later, but I have a bool g_bEnableMap = false; at the beginning of the plugin or get that wrong. Isn't that bool supposed to put g_bEnableMap false by default

By default yes, however plugins don't reload after map change unless they are modified. So you'll either want to set it false before checking on the next map again or set it to false with OnMapEnd.

[PinHeaDi]

So if I put g_bEnableMap = false; before checkmap(); in OnMapStart and not the in the checkmap();, it should work too

[Mitchell]

Originally Posted by PinHeaDi So if I put g_bEnableMap = false; before checkmap(); in OnMapStart and not the in the checkmap();, it should work too

sure, but it makes no difference what so ever where you put it either in checkmap() or in OnMapStart(), it's just for when reading it later instead of the variable be spread around the plugin you can have it where you check the maps list