[DoPe^]

really awesome plugin! testing it on our server atm.
and really gj on the sourcebans page edits also!

[ilga80]

Приложи в файле автор
Put in file

PHP Code:

CREATE TABLE `sb_comms` (
`bid` int(6) NOT NULL AUTO_INCREMENT,
`authid` varchar(64) NOT NULL,
`name` varchar(12 NOT NULL DEFAULT 'unnamed',
`created` int(11) NOT NULL DEFAULT '0',
`ends` int(11) NOT NULL DEFAULT '0',
`length` int(10) NOT NULL DEFAULT '0',
`reason` text NOT NULL,
`aid` int(6) NOT NULL DEFAULT '0',
`adminIp` varchar(32) NOT NULL DEFAULT '',
`sid` int(6) NOT NULL DEFAULT '0',
`RemovedBy` int( DEFAULT NULL,
`RemoveType` varchar(3) DEFAULT NULL,
`RemovedOn` int(11) DEFAULT NULL,
`type` tinyint(4) NOT NULL DEFAULT '0' COMMENT '1 - Mute, 2 - Gag',
`ureason` text,
PRIMARY KEY (`bid`),
KEY `authid` (`authid`),
KEY `created` (`created`),
KEY `RemoveType` (`RemoveType`),
KEY `type` (`type`),
KEY `sid` (`sid`),
KEY `aid` (`aid`),
FULLTEXT KEY `authid_2` (`authid`),
FULLTEXT KEY `name` (`name`),
FULLTEXT KEY `reason` (`reason`)
) ENGINE=MyISAM DEFAULT CHARSET=utf8; 


потому что смайлы мешают и невозможно создать таблицу
that smiles interfere

[ppalex]

Originally Posted by ilga80 Приложи в файле автор Put in file потому что смайлы мешают и невозможно создать таблицу that smiles interfere

Fixed/Исправил

[ocwoody]

Nice job with this! Will have to give it a try!

[DontWannaName]

What happens when we have to go and update Sourcebans?

[ppalex]

Originally Posted by DontWannaName What happens when we have to go and update Sourcebans?

In best case, you will have to re-edit the files, in the worst case (changes in sourcebans internal structure) you will have to wait until I release the adaptation to the new version.

[KorDen]

Originally Posted by ppalex I do not understand the meaning of integration basecomm in my plugin. functionality of one or two plug-in aggregate - the same Also I think, that replacement the default plugin is a bad thing for most of admins.

Why bad? Look at SB, sourcebans.smx unload basebans.smx, because when you using SB you don't need basebans functions. This plugin also provide full compatiblty with basecomms at console commands etc.

Spoiler

Почему ты думаешь, что полная интеграция basecomm в твой плаги и отказ от старого это плохо? Посмотри на принцип SB - ведь плагин basebans выгружается и отключается при запуске SB, поскольку его функции не нужны. Так и тут, если фактически твой плагин копирует функциональность обычного, а так же расширяет ее, то зачем нужен обычный плагин вообще? Им просто никто не будет пользоваться. Старую функциональность sm_gag и прочих ты уже сохранил, поскольку два других параметра опциональны. В итоге никакого дискомфорта не ощущается

[ppalex]

Originally Posted by KorDen Spoiler Почему ты думаешь, что полная интеграция basecomm в твой плаги и отказ от старого это плохо? Посмотри на принцип SB - ведь плагин basebans выгружается и отключается при запуске SB, поскольку его функции не нужны. Так и тут, если фактически твой плагин копирует функциональность обычного, а так же расширяет ее, то зачем нужен обычный плагин вообще? Им просто никто не будет пользоваться. Старую функциональность sm_gag и прочих ты уже сохранил, поскольку два других параметра опциональны. В итоге никакого дискомфорта не ощущается

Spoiler

Потому что мой плагин далеко не всё копирует, и с банами несколько проще - оригинальный плагин банов предоставляет другим плагинам не так много функциональности.
Мой же плагин тесно использует basecomm - обращаясь к его внутренним вызовам вроде IsClientGagged или SetClientGag. Кроме того, реализацией отключенного чата или микрофона - занимается не мой плагин, а basecomm. Также при интеграции basecomm в мой плагин - придется переписывать basecomm.inc, и думаю что при распаковке, многих насторожит предложение перезаписывать базовый файл SourceMod

В принципе, я думал над их объединением, но 1) я подожду пока проверят плагин (потому что подозреваю что там еще есть что исправлять) 2) замену плагина basecomm буду делать опциональной (то есть к теме будет приложено 2 .smx файла. заменяющий и нет)

Basebans doesn't provide any natives or forwards like as basecomms. Also, my plugin doesn't implement "locking" of chat or voice - it implemented in basecomms.
I will wait review of plugin and then i will think about full replacement of basecomm. Will be necessary to consult with plugin approvers

[ppalex]

I remembered that the biggest problem is that my plugin should be named basecomm.smx, otherwise it will not be compatible with all other plugins, which use the original basecomm.smx

[Root_]

Originally Posted by ppalex I remembered that the biggest problem is that my plugin should be named basecomm.smx, otherwise it will not be compatible with all other plugins, which use the original basecomm.smx

please explain me why that? Usually plugins is dealing with each other using natives, not plugin name

Spoiler

плагины, которые используют функцию FindPluginByName("имя.smx"), здесь не одобряются (если мне не изменяет память)