Русское сообщество Chinascript.ru по поддержке и русской локализации Discuz!, ECShop, ECMall, UCenter

 Забыли пароль?
 Регистрация
Поиск
Популярное: Discuz ECMall ECShop
Автор: admin

[CMS] CMS DigitalMallCenter

   Закрыть [Скопировать ссылку]
Опубликовано 21-07-2016 08:40:32 | Показать все сообщения
Наши разработки
сообщение изменено Cripter в 21-07-2016 08:49
Новый день новые предложения - если позволите
Я не хакер но обезопасить стоит..........

1. Сделайте также проверку на ЗАПРЕТ ввода наклонных / и \  в названии "Добавления перевода"


Выводится ошибка
Обратите внимание на ВЕРТИКАЛЬНЫЙ ползунок - наверное ошибка идет с повторением до бесконечности просто через некоторое время страница обновляется.


2. Если получится (да и вообще это одно из главных условий файла переводов) - ДОЛЖНА быть УНИКАЛЬНАЯ переменная со своим текстом перевода.
А то получится абракадабра и как системе понять какую переменную выводить если в файле несколько ОДИНАКОВЫХ переменных?


Пункт 2 не столь серьезный и это в целях самого администратора иметь рабочий сайт с нормальными переводами, поэтому это можно отнести на совесть администратору.
Но если тукую проверку можно сделать то это будет просто супер - можно избавиться от "небольшой" головной боль и переводе и одинаковых переменных особенно если их за несколько сотен .

Так что это на Ваше рассмотрение оставлять как есть или добавить условие при Создании переменной и при редактировани на Одинаковость.

Комментарий

Особой валидации не делал, так как смысла нет, это же утилите для админа. Делал только там, где это действитель  Подробная информация Ответить Опубликовано 21-07-2016 14:20
1. в названии "Добавления перевода" сделал запрет на все символы, кроме a-z, 0-9 и нижнего подчеркивания. Насчет о  Подробная информация Ответить Опубликовано 21-07-2016 14:04
 Автор| Опубликовано 21-07-2016 14:04:13 | Показать все сообщения
добавил Cripter в 21-07-2016 09:40
сообщение изменено Cripter в 21-07-2016 08:49
Новый день новые  ...

1. в названии "Добавления перевода" сделал запрет на все символы, кроме a-z, 0-9 и нижнего подчеркивания.

Насчет ошибки не понял. При каких условиях она происходит?

2. Уникальность переменной (ключа) проверяется только при первоначальном добавлении. Проверка проходит рекурсивно по всем языковым файлам, если такой ключ уже имеется, выводится соответствующее сообщение и скрипт останавливается.
Не советую добавлять новые ключи через редактирование файла, так как возможны такие накладки.
На заметку: при одинаковых ключах ошибки не будет, просто при нескольких одинаковых ключах (у них могут быть разные значения) будет выведено значение с последнего ключа. Ну а если хотите добавлять - тогда проверьте сначала ключ на уникальность через поиск по фразам, но я бы советовал, все-таки, добавлять перевод через соответствующий функционал Добавить перевод (он специально и сделан для того, чтобы проверять ключи на уникальность).

 Автор| Опубликовано 21-07-2016 14:20:46 | Показать все сообщения
добавил Cripter в 21-07-2016 09:40
сообщение изменено Cripter в 21-07-2016 08:49
Новый день новые  ...

Особой валидации не делал, так как смысла нет, это же утилите для админа. Делал только там, где это действительно необходимо, ну как в данном случае, чтобы по ошибке самому не вставить не нужный символ, который может поломать код.
Опубликовано 21-07-2016 17:28:51 | Показать все сообщения
Насчет ошибки не понял. При каких условиях она происходит?

1. Вы уже все подправили - заново не получается проверить.

2. На счет проверки - все нормально странно что утром такое можно было провернуть (на скринах вверху видны одинаковые переменные) - сейчас выдает нужные сообщения об ошибках и создать новый перевод с таким же именем не получается. Иными словами - все работает как часики.

- Лишние символы ввести нельзя
- Создать переменную (имя ключа) с там же именем нельзя
- В название ключа ввести русские символы нельзя
- В название ключа ввести ТОЧКУ наклонную и другие символы нельзя

Вроде все - Вы просто молодец, оперативно и качественно все работает




Комментарий

Это Вам Спасибо! Все это было сделано с Вашей помощью! Еще несколько дней и будем переезжать на нашу новую пр  Подробная информация Ответить Опубликовано 21-07-2016 18:11

Рейтинг

Количество участников 1Золото +5 Магия +3 Свернуть Инфо
admin + 5 + 3 Великолепно!

Показать список

 Автор| Опубликовано 21-07-2016 18:11:17 | Показать все сообщения
добавил Cripter в 21-07-2016 18:28
1. Вы уже все подправили - заново не получается провери ...

Это Вам Спасибо! Все это было сделано с Вашей помощью!
Еще несколько дней и будем переезжать на нашу новую прописку. Готовим сайт и форум. Там будут только наши разработки, в том числе и текущие.
Опубликовано 26-07-2016 11:04:49 | Показать все сообщения
Раз тут от Вас тихо - значить работы по программному коду проводиться много - это радует.

Готовим сайт и форум.

Может хоть простой landing page + форум

Комментарий

Да, спасибо. Сделали сайтик и форум. Форум вообще без дефолту запустили, потом уже накинем стиле. Так что мы уж  Подробная информация Ответить Опубликовано 26-07-2016 18:13
 Автор| Опубликовано 26-07-2016 18:13:56 | Показать все сообщения
добавил Cripter в 26-07-2016 12:04
Раз тут от Вас тихо - значить работы по программному ко ...

Да, спасибо. Сделали сайтик и форум. Форум вообще без дефолту запустили, потом уже накинем стили. Так что мы уже здесь: https://jqsee.com
 Автор| Опубликовано 27-07-2016 09:48:02 | Показать все сообщения
Тема закрыта. Огромную благодарность за помощь в поиске багов выражаем участнику Cripter! Продолжать обсуждение данного продукта будем на форуме разработчиков https://jqsee.com/forum/viewforum.php?f=3
Всем, Удачи!




Чтобы ответить, вам надо авторизироваться в системе Вход | Регистрация

Правила начислений

Закрыть

Горячая информацияНазад /2 Вперед

Карта|Теги|Черный список|Версия для КПК|Архив|RSS|Русское сообщество Chinascript.ru

GMT+2, 23-09-2017 14:45 , Processed in 0.037268 second(s), 16 queries , Gzip On.

Powered by Discuz! X3.1 Licensed

© 2001-2013 Comsenz Inc.

Быстрый ответ Вернуться к началу Назад к списку