Смс уведомления

Запрос улучшений, интеграции с различными системами и т.д.
Аватара пользователя
rootden
Сообщения: 260
Зарегистрирован: 2010-09-24 8:28:44
Благодарил (а): 1 раз

Re: Смс уведомления

Сообщение rootden » 2013-05-21 23:40:51

все, ок зашел

Аватара пользователя
Alex Keda
Сообщения: 2882
Зарегистрирован: 2009-10-07 14:30:54
Откуда: USSR
Поблагодарили: 19 раз

Re: Смс уведомления

Сообщение Alex Keda » 2013-05-26 18:14:51

hosts/billing/config/Config.ini

Код: Выделить всё

+SMSGateway="SMS шлюз"
+Provider="Провайдер"
+Key="Ключ доступа"
+Interval="Интервал (сек)"
+lowBalance="Предупреждение"
+Sender="Отправитель"
+Price="Стоимость сообщения"
+ru="Для России"
+ua="Для Украины"
+sng="Для СНГ"
+zone1="Зона 1"
+zone2="Зона 2"
+default="Прочее"
+Exceptions="Исключения (free стоимость)"
+LimitPaidInvoices="Сумма оплаченных счетов"
+LimitSchemeID="ID тарифов (через запятую)"

этта песдетц, простите за мой франзцузский =))

1. config.ini - он общий. и если понадобится код россии ещё где-то - он везде будет "для россии"... а не "Россия"
2. используй префиксы к названиям пунктов, т.к. вот занял ты ходовые слова типа default, Interval, ... прежде чем делать такое, стоит задуматься - а почему они до сих пор не заняты ни в каком конфиге? биллинг алеко не маленькйи, ниде ни дефолта, ни интервала... ничё не было чтоле?
3. если надо описание - для этого есть Prompt.ИмяПункта="Описание чё за пункт ИмяПункта" не надо лепить в пункт описание, в скобках или прям текстом сплошным...

и это был первый файл для которого переводы строк привёл в порядок, чтоб diff можно было сделать...
Убей их всех! Бог потом рассортирует...

Аватара пользователя
Alex Keda
Сообщения: 2882
Зарегистрирован: 2009-10-07 14:30:54
Откуда: USSR
Поблагодарили: 19 раз

Re: Смс уведомления

Сообщение Alex Keda » 2013-05-26 18:25:51

2. твой редактор после запятой обязательно пробел ачем-то добавляет
это "взрывает" файлы нахер, при сравнении...
вот например:

Код: Выделить всё

-$Foto = GetUploadedFileSize('Users',$__USER['ID']);
-#-------------------------------------------------------------------------------
-$Comp = Comp_Load('Upload','UserFoto',$Foto?SPrintF('%01.2f Кб.',$Foto/1024):'не загружена');
-if(Is_Error($Comp))
-  return ERROR | @Trigger_Error(500);
-#-------------------------------------------------------------------------------
-$Table[] = Array('Персональная фотография (90 x 110)',$Comp);
-#-------------------------------------------------------------------------------
-if($Foto){
-  #-----------------------------------------------------------------------------
-  $Comp = Comp_Load(
-    'Form/Input',
-    Array(
-      'type'  => 'checkbox',
-      'name'  => 'IsClear',
-      'value' => 'yes'
-    )
-  );
-  if(Is_Error($Comp))
+$Foto = GetUploadedFileSize('Users', $__USER['ID']);
+#-------------------------------------------------------------------------------
+$Comp = Comp_Load('Upload', 'UserFoto', $Foto ? SPrintF('%01.2f Кб.', $Foto / 1024) : 'не загружена');
+if (Is_Error($Comp))
     return ERROR | @Trigger_Error(500);
-  #-----------------------------------------------------------------------------
-  $Table[] = Array(new Tag('SPAN',Array('style'=>'cursor:pointer;','onclick'=>'ChangeCheckBox(\'IsClear\'); return false;'),'Удалить фотографию'),$Comp);
+#-------------------------------------------------------------------------------
+$Table[] = Array('Персональная фотография (90 x 110)', $Comp);
+#-------------------------------------------------------------------------------
+if ($Foto) {
+    #-----------------------------------------------------------------------------
+    $Comp = Comp_Load(
+       'Form/Input', Array(
+   'type' => 'checkbox',
+   'name' => 'IsClear',
+   'value' => 'yes'
+       )
+    );
+    if (Is_Error($Comp))
+   return ERROR | @Trigger_Error(500);
+    #-----------------------------------------------------------------------------
+    $Table[] = Array(new Tag('SPAN', Array('style' => 'cursor:pointer;', 'onclick' => 'ChangeCheckBox(\'IsClear\'); return false;'), 'Удалить фотографию'), $Comp);
 }

найди хоть одно логическое отличие? а код вообще другой...
не надо умных редакторов, надо просто чтоб подсвечивали тебе, а не правили код за тебя в непойми что...

в общем я не знаю как это коммитить...
конфиги, например, оно у тебя вообще перелохматило - пробеллы все на табы заменило, дифф показывает 100% отличий между старым и новым файлом, хотя умом я понимаю, что ты туда не болше 10 строк внёс...
Убей их всех! Бог потом рассортирует...

Аватара пользователя
Alex Keda
Сообщения: 2882
Зарегистрирован: 2009-10-07 14:30:54
Откуда: USSR
Поблагодарили: 19 раз

Re: Смс уведомления

Сообщение Alex Keda » 2013-05-26 18:27:24

а это вообще треш какой-то...

Код: Выделить всё

+
 #-------------------------------------------------------------------------------
-/** @author Великодный В.В. (Joonte Ltd.) */
-/******************************************************************************/
-/******************************************************************************/
+/** @author Rootden for Lowhosting.ru */
+/* * *************************************************************************** */
+/* * *************************************************************************** */
 $__args_list = Array('Task', 'Mobile', 'Message', 'ID');
-/******************************************************************************/
+/* * *************************************************************************** */

ну нахрена?

не автора, заметь нахрена сменил, а нахрена разделители поменял?
Убей их всех! Бог потом рассортирует...

Аватара пользователя
Alex Keda
Сообщения: 2882
Зарегистрирован: 2009-10-07 14:30:54
Откуда: USSR
Поблагодарили: 19 раз

Re: Смс уведомления

Сообщение Alex Keda » 2013-05-26 18:50:04

чё делать-то будем?
если в таком виде коммитить - бред какой-то будет...
надо причёсывать до состояния нормального

либо коммитим, релизим - уже на неделе релиз будет, дальше причёсываешь...
смотри, короче...
Убей их всех! Бог потом рассортирует...

Аватара пользователя
Alex Keda
Сообщения: 2882
Зарегистрирован: 2009-10-07 14:30:54
Откуда: USSR
Поблагодарили: 19 раз

Re: Смс уведомления

Сообщение Alex Keda » 2013-05-26 20:02:07

могу дать то что щас в репозитории, или сделаем релиз - обновишь до релиза
а потом аккуратно, поправить разницу

но с переменными надо решать конечно.
Убей их всех! Бог потом рассортирует...

Аватара пользователя
rootden
Сообщения: 260
Зарегистрирован: 2010-09-24 8:28:44
Благодарил (а): 1 раз

Re: Смс уведомления

Сообщение rootden » 2013-05-26 22:54:17

давай то что есть, я отключил авто форматирование кода для всего файла, только для измененного оставил.

Аватара пользователя
Alex Keda
Сообщения: 2882
Зарегистрирован: 2009-10-07 14:30:54
Откуда: USSR
Поблагодарили: 19 раз

Re: Смс уведомления

Сообщение Alex Keda » 2013-05-26 23:45:27

в аттаче инсталлер версии из репозитория, на данный момент (последнее изменение - автообновление окна тикетницы)
Вложения
JBs.tar.gz
(3.1 МБ) 334 скачивания
Убей их всех! Бог потом рассортирует...

Аватара пользователя
Alex Keda
Сообщения: 2882
Зарегистрирован: 2009-10-07 14:30:54
Откуда: USSR
Поблагодарили: 19 раз

Re: Смс уведомления

Сообщение Alex Keda » 2013-05-26 23:47:38

вообще, проще будет тебе дать доступ в репозиторий...
но уже после релиза, думаю...
т.к. щас просто по незнанию можешь чёнить запороть из сделанного.
а после - всегда можно откатить да и время до след. релиза будет...
Убей их всех! Бог потом рассортирует...

Аватара пользователя
Alex Keda
Сообщения: 2882
Зарегистрирован: 2009-10-07 14:30:54
Откуда: USSR
Поблагодарили: 19 раз

Re: Смс уведомления

Сообщение Alex Keda » 2013-05-27 23:23:31

инсталлер на тока что, с последними правками
Вложения
JBs.tar.gz
(3.11 МБ) 311 скачиваний
Убей их всех! Бог потом рассортирует...


Вернуться в «Запрос улучшений»

Кто сейчас на конференции

Сейчас этот форум просматривают: нет зарегистрированных пользователей и 13 гостей