server-beta icon indicating copy to clipboard operation
server-beta copied to clipboard

Some suggestions...

Open Zorono opened this issue 2 years ago • 1 comments

Description

it'd be great if you implemented such functions as

YSF plugin (which is no longer compatible with OMP):

native ffind(const pattern[], filename[], len, &idx);
native frename(const oldname[], const newname[]);
native dfind(const pattern[], filename[], len, &idx);
native dcreate(const name[]);
native drename(const oldname[], const newname[]);

which are very useful for File&Directory management on Ownership systems like Houses&Vehicles&Bussiness which are based on INI files... and such callback

forward OnRemoteRCONPacket(const ipaddr[], port, const password[], success, const command[]);

which is very useful for monitoring RCON's usage

KashCherry's timerfix plugin (which is no longer compatible with OMP):

native IsValidTimer(timerid);
native PauseTimer(timerid);
native ContinueTimer(timerid);
native GetTimerRemainingTime(timerid);
native GetCountOfRunningTimers();

which are very useful for Timer management like Validating timers before killing them using IsValidTimer...

  • Please do not ignore these suggestions and I hope to accept them, please

Contact information

Discord: [BR]John_Magdy#8791

Zorono avatar Sep 17 '22 12:09 Zorono

// cons

As for releasing, from my point of view this code is frankly no-where near releasable. I keep looking at it wondering what you lot are doing. The old version was taking a while (though would have been done by now had it had half the effort put in that this one has), but at least it wasn't frankly awful. This needs a high-level review, because it completely misses every point of doing open.mp in the first place

Hual avatar Jan 05 '22 18:01 Hual

True, but add // cons first.

Alasnkz avatar Jan 05 '22 18:01 Alasnkz