Language-based date display format support

[michaelkl]

Алексей, проверьте пожалуйста работоспособность функции "Добавить цель" на вкладке Планирование. Я добавляю цель, заполняю поля, но когда нажимаю "Сохранить запись" выскакивает ошибка в cash/ajax/save_goal.php. Видимо, что-то я не так туда передаю, но вот что - не могу понять.

[pihel]

Алексей, проверьте пожалуйста работоспособность функции "Добавить цель" на вкладке Планирование.

Посмотрите в логах ошибку, подозреваю, что обновление прошло некорректно и не создалась таблица ( https://github.com/pihel/cash/blob/master/lib/update.php#L57 ). Сейчас проще всего ее создать вручную любым редактором sqlite или через пхп..

Михаил, предлагаю все же не плодить глобальных переменных "langDateFormat ". Предлагаю добавить новый ключ в js переменную "settings" ( https://github.com/pihel/cash/blob/master/lib/init.php#L84 ) и использовать ее в js файлах.

[michaelkl]

langDateFormat переделал на settings.date_format.

Ошибку пока не победил, в логе: PHP Fatal error: Call-time pass-by-reference has been removed in /var/www/cash/ajax/save_goal.php on line 6 Таблица создана нормально. Завтра буду дальше думать.

[pihel]

Включите режим дебага тут https://github.com/pihel/cash/blob/master/lib/init.php#L8 и просмотрите полный лог выполнения.

И еще посмотрите, точно ли колонки таблицы соответствуют create table?

[michaelkl]

Алексей, я покопался и вот что обнаружил. Даже без моих добавлений эта штука в моей системе не работает. Похоже, всё дело в PHP 5.6 или каких-то его настройках. Вот что пишут про эту ошибку: http://stackoverflow.com/questions/8971261/php-5-4-call-time-pass-by-reference-easy-fix-available.

Я попытался внести исправления как пишут на stackoverflow, ошибка пропала, но теперь цель не добавилась.

Я думаю, нужно вам потестировать у себя мой код, и если с ним всё хорошо, то этот пул-реквест закрываем, а по поводу ошибки заведём новый issue.

[pihel]

Я попытался внести исправления как пишут на stackoverflow, ошибка пропала, но теперь цель не добавилась.

Вы убрали тут https://github.com/pihel/cash/blob/master/ajax/save_goal.php#L6 амперсант? И все равно ошибка осталась?

[michaelkl]

Да, я убрал амперсанд, ошибка пропала, но цель в таблицу не добавилась

[pihel]

Да, я убрал амперсанд, ошибка пропала, но цель в таблицу не добавилась

У меня, к сожалению, нет под рукой php 5.6 для проверки, на более старых версиях работает без ошибок.

По поводу реквеста: если уж менять формат даты, то, наверно, надо это сделать во всех местах. Сейчас нет времени доделать код.

[michaelkl]

Алексей, я вроде бы во всех местах поменял. Скажите, если где-то увидели, что я пропустил, я сам доделаю. Там в некоторых местах по коду, если я правильно понял, формат устанавливается для передачи в БД, поэтому там я не трогал.

[pihel]

А, это может быть, надо будет внимательней мне посмотреть.

[pihel]

Извиняюсь за долгое ожидание, все времени не было посмотреть. Все хорошо, действительно формат y-m-d остался только при передаче параметров.

[pihel]

Внес небольшие изменения, а то товар не добавлялся, почему то.