SB-MaterialAdmin / Web

SourceBans Material Design
64 stars 45 forks source link

Сохранение пароля под звездочками при редактировании админа #299

Open bigbrain911 opened 4 years ago

bigbrain911 commented 4 years ago

Пожалуйста реализуйте, чтобы при редактировании админа, все 3 поля с паролем сохранялись, но под звездочками, т.е. чтобы не приходилось заново вводить данные (вроде как этого было ранее достаточно {else} value="{$a_serverpass}")

В сб от Онотоле было это реализовано, возможно кусок кода упростит задачу:

код.txt

megoRU commented 4 years ago

Либо я тупой либо да. я так понял вы про это.

Если я хоть что-то понимаю в php и в tpl. Это вроде логика в php, а я в нем 0. Я джавист (не андроид)

то что вы скинули это код для tpl

У меня такое чувство что эта фича не для моих знаний

image

megoRU commented 4 years ago

Крч это в php. Скинь его гитхаб. tpl что его что крузи так бегло глянул что 100% все так же

bigbrain911 commented 4 years ago

Да я как бы не сильно разбираюсь в коде, просто ранее поднимал тему на форуме, обещали внедрить и все забылось, смысл в том что в данный момент если поменять админу любой из параметров, то страница открывается с пустыми паролями, т.е. если сохранить админа в таком виде то пароль на сервер не будет установлен и это неприемлемо.

megoRU commented 4 years ago

крч скинь его гитхаб, я утром чекну и напишу что получилось что нет.

bigbrain911 commented 4 years ago

Эти наработки остались только у меня на диске, я закину .tpl и .php

megoRU commented 4 years ago

Окей

bigbrain911 commented 4 years ago

Новая папка.zip

megoRU commented 4 years ago

@bigbrain911 Так там же проверка. Если поле пустое тогда не будет обновлять.

Если ты хочешь чтобы пароль был под звездочками. То моих знаний тут недостаточно

image

bigbrain911 commented 4 years ago

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

megoRU commented 4 years ago

Понял. Посмотрю что смогу сделать. Надо тогда проверку сделать как на обычном логин/пароль

bigbrain911 commented 4 years ago

Я давал код там это было уже реализовано.

megoRU commented 4 years ago
  1. php не моё)
  2. В том коде что ты скинул это не реализовано.
  3. Теперь нельзя удалить пароль через web. Нужно вручную лезть в БД ибо я не придумал костыль (чекбокс для удаления сделать я могу, но это работы на часок, ибо там нужно еще шаблон править)
  4. Теперь пароль не меняется при сохранении. Если чекбокс выключен.
  5. Не реализовал твою просьбу запихнуть в звездочки (смотреть пункт 0 )

В ветку кидать не буду. Так как это костыль. А так можешь юзать у себя локально

image

Пофиксил admin.edit.admindetails.zip

bigbrain911 commented 4 years ago

Я скачал последнее обновление файла, все работает как нужно, но проблемы с визуализацией, сейчас после обновы так:

1111111

А должно быть так:

22222222

Т.е. юзер он же админ, визуально не понимает установлены ли пароли на сайте и на сервере.

megoRU commented 4 years ago

Я понимаю, но это не в моей компитенции)) Вот когда @CrazyHackGUT найдет время он быстро пофиксит это)

bigbrain911 commented 4 years ago

К сожалению у него постоянно нет времени, понятно что это хобби, работа есть работа.

megoRU commented 4 years ago

@bigbrain911 @CrazyHackGUT (позаимствовал с admin.edit.server.php + tpl логику)

Доделал.

Что теперь:

  1. Если есть пароль на сервер у админа то тогда поле == Скрыт
  2. Если нет пароля значит пустое
  3. Если нужно удалить пароль. Делаем пустое и выключаем тумблер и сохраняем

Все что нужно.zip

bigbrain911 commented 4 years ago

А можно сделать чуть по другому, чтобы все пароли были под звездочками если они уже были установлены пункт 1 и 2 на скрине, а так же если установлен пароль на сервер то ползунок в пункте 3 должен быть смещен вправо и гореть зеленым. Мне кажется так было бы нагляднее и правильнее.

фикс

bigbrain911 commented 4 years ago

А так же в старом сб от Онотоле было видно срок админки, опять же если возможно было бы правильно чтобы писало срок а если навсегда то ползунок смещен право и горит зеленым, это было бы очень круто и понятно даже для новичков.

megoRU commented 4 years ago

1 и 2 не по design. Вообще не принято посылать пароль на поле ввода из БД. Даже если заменять на Скрыт

На счет тумблера подумаю