Skip to content

Сделать как минимум проверку Check nonce #23

@BootWP

Description

@BootWP

При работе через ajax, нужно обязательно хотя бы пройти проверки Check nonce, иначе сайт очень уязвим для взлома.
Подробнее как работать с ajax описано здесь https://wp-kama.ru/id_2018/ajax-v-wordpress.html

Также можно смотреть как сделано у нас уже для бакенда.

  1. Идет запрос через ajax, по мимо других переменных в нем добавляются и название функции php и nonce
    image

  2. Далее в php ajax проверяется nonce
    image

  3. Здесь не понятно какие данные попадут в file_get_contents('php://input') лучше за ранее знать какие параметры приходят в каком формате и также почистить хотябы както до добавления в базу.
    image
    если приходит json то не нужно пройти через json_decode, можно json сразу в базе сохранить.

Стили не добавлять прямо в теги, только в очень крайних случаях когда без этого некуда,
image

добавить все в /assets/admin.css или для front style.css и использовать классы с префиксом mcwallet-
image

Ну и так как нужно стараться командой держать один стиль написания кода,
Пока только три моменты.

  1. Названия файлов не нужно camelCase userPanel.php а user-panel.php

  2. Всегда хуки и фильтры добавляем после функции не перед ней.
    image

  3. Открывающие фигурные скобки функции не переносим в новую строку а оставляем в тотже ряд что и функция.
    image

Пока на данный момент все, если будут вопросы пишите.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions