#nulldvachin #1

lisanyan · 2016-06-20T13:03:20Z

I suck at intros..

В общем, если файл загружен на pomf.cat, то src получается вида
//a.pomf.cat/plapla.webm (линк без протокола)
и в итоге аппендится к домену борды.
В массиве файла есть ключ external_upload (1/0), можно по нему проверять, и не аппендить линк или добавлять к нему протокол (e.g. "https:" + //a.pomf.cat/plapla.webm).

Еще в апи пару новых фич добавила
Удаление постов теперь может отдавать json, параметр ajax=1 в запрос

{
   "error" : {
      "1325" : "Неверный пароль для удаления.",
      "1327" : "Неверный пароль для удаления.",
      "1292" : "Неверный пароль для удаления.",
      "766" : "Неверный пароль для удаления."
   },
   "error_code" : 200
}

При успешном удалении вернет
{ "redir" : "/b/thread/num" }
или если был удален тред
{ "redir" : "/b/" }

Еще желательно добавлять параметр parent с номером треда к delete task, если пост из треда удаляется, чтобы правильный редирект был.

Список борд теперь можно забирать по /board/api/getboards

Алсо, теги sup/sub в разметку можно добавить и инпут no_pomf (не грузить файлы на внешний сервер) в форму отправки (если не сложно).

The text was updated successfully, but these errors were encountered:

Mishiranu · 2016-06-20T14:39:25Z

Вроде почти всё сделал, кроме двух вещей.

Во-первых, мне не очень нравится апи списка досок т. к. они не упорядочены. Правильно отдавать массив объектов с полями "board_name" и "title", например. Или на сервере в принципе не предусмотрен порядок досок, а на страницах доски вбиты вручную?

Во-вторых, поддержку "no_pomf" добавить не получится. Модули тут бессильны, новые кнопочки-галочки можно добавлять только на стороне приложения. Я бы подумал насчёт добавления в приложение, но фича уж больно уникальная и, по моему мнению, не сильно востребованная.

lisanyan · 2016-06-20T14:50:25Z

Массив сейчас переделаю с сортировкой и ключами board_desc/board_name. На страницах вручную забиты, руки никак не дойдут допилить динамическую подгрузку досок.

Окей, no_pomf - не критично.

Mishiranu · 2016-06-20T14:59:17Z

Так доски по алфавиту сортируются? Просто сортировать json-объект нет никакого смысла, потому что он в принципе не может быть упорядоченным, в любом случае стоит сделать массив.

lisanyan · 2016-06-20T15:02:44Z

Да. Действительно, но пусть будет, ибо мне один аргумент достаточно было json энкодеру скормить.

Готово, сейчас нормально?

Mishiranu · 2016-06-20T15:07:37Z

У меня что-то сайт вообще отвалился.

lisanyan · 2016-06-20T15:13:33Z

Сейчас вроде открывается, у хостера что-то с IO, диски оче медленно работают.

Mishiranu · 2016-06-20T15:18:00Z

Так что, думаешь объектом всё таки оставить? А то я что-то не вижу разницы по сравнению с прошлым разом, лол.

lisanyan · 2016-06-20T15:23:10Z

Не. Т.е. мне просто data массивом сделать?

Mishiranu · 2016-06-20T15:27:50Z

Да, всё верно. И к каждому элементу массива добавить ещё поле кода доски.

lisanyan · 2016-06-20T15:54:02Z

Интернеты отвалились :\

Что-то вроде

   "data" : [
      {
         "board_desc" : "Чятик для двоих",
         "board_key" : "a",
         "board_name" : "Аниме"
      },
    ]

получилось

lisanyan · 2016-06-20T15:55:29Z

Или так сделать?

   "data" : [
      "a" : {
         "board_desc" : "Чятик для двоих",
         "board_key" : "a",
         "board_name" : "Аниме"
      },
    ]

Mishiranu · 2016-06-20T15:57:12Z

Первый вариант лучше. Короче, как сейчас — лучше всего.

lisanyan · 2016-06-20T15:58:41Z

Done.

Mishiranu · 2016-06-20T16:02:37Z

Ок, тогда я всё сделал, сейчас обновлю.

lisanyan · 2016-06-20T16:03:38Z

Сек

lisanyan · 2016-06-20T16:03:59Z

Я думаю возврат апи удалялки тоже массивом сделать

Mishiranu · 2016-06-20T16:04:48Z

Да это уже необязательно.

lisanyan · 2016-06-20T16:07:07Z

Ну ладно, тогда как есть оставлю. Спасибо.

Mishiranu · 2016-06-20T16:08:08Z

Просто я уже закоммитил релиз, лол. Я могу откатить, конечно, просто стоит ли?

lisanyan · 2016-06-20T16:19:56Z

Ну если его легче парсить будет, то думаю да. Или инкремент версии сделать до 1.4 лучше

Алсо, delete еще это выдать может,

Тред закрыт
Неправильный IP - (посты можно удалять без пароля, если айпи клиента совпадает с ip поста)
Пост протух.

Или необязательно хандлить все сообщения?

Mishiranu · 2016-06-20T16:23:00Z

Ну, мне разницы-то нет, но если хочешь сделать чтобы типа "нормально" было — давай. Выпущу как 1.4 потом.

Обработку этих сообщений тоже добавлю.

Mishiranu · 2016-06-20T16:28:01Z

Только лучше не так, как сейчас

{
   "error" : [
      {
         "380" : "Неверный пароль для удаления."
      }
   ],
   "error_code" : 200
}

а как-то так например:

{
   "error" : [
      {
         "post_id": "380",
         "reason": "Неверный пароль для удаления."
      }
   ],
   "error_code" : 200
}

Mishiranu · 2016-06-20T16:30:58Z

Кстати, вообще для апи рекомендую Content-Type отдавать application/json; charset=utf-8 вместо простого application/json.

lisanyan · 2016-06-20T16:34:26Z

Оке, просто переписывать историю коммитов не есть хорошо, если форки есть :D
Теперь такой вывод (наверное еще больше complicated :\ )

"error" : [
      {
         "error_msg" : "Неверный пароль для удаления.",
         "post" : "1331"
      },
   ],

Поправила Content-Type

lisanyan · 2016-06-20T16:38:08Z

Теперь post_id/reason в объекте

lisanyan · 2016-06-20T16:48:15Z

Алсо, клиент куки сохраняет которые с ответом сервера после постинга прилетели?

Mishiranu · 2016-06-20T16:49:02Z

Нет. Что за куки и для чего они?

lisanyan · 2016-06-20T16:57:01Z

name, если в нейме есть трип который есть в дб, то можно постить на любых досках с неймом и без капчи (что-то вроде пасскодов, лол).

Mishiranu · 2016-06-20T16:58:20Z

Не понял, а почему эта кука после постинга появляться должна?

Mishiranu · 2016-07-14T21:29:39Z

А, да, текст должен быть коротким и влезать в 1 строку. Оно на что-то более серьёзное не было рассчитано.

Mishiranu · 2016-07-14T21:34:12Z

Алсо, функция поста не всегда отдает поле banned, не будет exception из-за того что его нет, или лучше добавить его во все ошибки?

int banned = jsonObject.optInt("banned", 0);

Не будет. Если оно отсутствует или оно не преобразуется в число — вернёт 0.

lisanyan · 2016-07-14T22:12:00Z

Тогда окей.

Поиск по борде еще можно прикрутить? /api/search
Параметры

find - строка для поиска
op - искать только в оп-постах
comment - поиск по тексту в посте (должен быть 1)
subject - поиск по теме

Mishiranu · 2016-07-14T22:25:40Z

Тоже готово.

lisanyan · 2016-07-14T22:29:30Z

Thx~

Обновляй.

lisanyan · 2016-07-15T20:30:29Z

#crash
В общем, если я отключаю флажки на борде, и рефрешу полностью тред (который закешировался с флажками), то клиент вылетает.

Device: Sony Ericsson hallon (MT15i)
SDK: 18 (4.3.1)
Application: 1031 (2.8.0)
----------------------------------------
java.lang.RuntimeException: An error occured while executing doInBackground()
    at android.os.AsyncTask$3.done(AsyncTask.java:299)
    at java.util.concurrent.FutureTask.finishCompletion(FutureTask.java:352)
    at java.util.concurrent.FutureTask.setException(FutureTask.java:219)
    at java.util.concurrent.FutureTask.run(FutureTask.java:239)
    at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1080)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:573)
    at java.lang.Thread.run(Thread.java:841)
Caused by: java.lang.NullPointerException
    at chan.content.model.Post.contentEquals(Post.java:559)
    at chan.content.model.Posts.pendingMerge(Posts.java:339)
    at com.mishiranu.dashchan.async.ReadPostsTask.doInBackground(ReadPostsTask.java:184)
    at com.mishiranu.dashchan.async.ReadPostsTask.doInBackground(ReadPostsTask.java:1)
    at android.os.AsyncTask$2.call(AsyncTask.java:287)
    at java.util.concurrent.FutureTask.run(FutureTask.java:234)
    ... 3 more

Mishiranu · 2016-07-15T20:38:16Z

Да, глупая ошибка. Попробуй удали пока что тред из кэша (или вообще кэш очисти), а я сейчас исправлю.

Mishiranu · 2016-07-15T20:38:36Z

Это ошибка не в модуле, а в клиенте, так что только с его обновлением что-то изменится.

lisanyan · 2016-07-15T20:44:13Z

Оки.

lisanyan · 2016-07-15T20:49:09Z

Done.

Mishiranu · 2016-07-15T20:49:56Z

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

Mishiranu · 2016-07-15T20:53:02Z

Да я уже всё проверил.
На будущее — лучше создай issue у самого проекта, если подобный вылет происходит, расширение само по себе не может приводить к вылету (нет, ну может конечно, если постараться), а если всё таки приводит — это баг клиента.

lisanyan · 2016-07-15T21:00:54Z

Получилось пофиксить?

Alrighty.

Mishiranu · 2016-07-15T21:04:15Z

Да, получилось.

lisanyan · 2016-07-22T12:13:29Z

API update

Новые поля в конфиге

max_comment_length //Maximum number of characters in a comment
max_field_length //Maximum number of characters in subject, name, and email
default_name //Дефолтный нейм (аноним итд)

Скорость борды (постов в час) - поле board_speed в boardinfo.

Печенька auth переименована в wakaauth

Алсо, категории борд можешь прикрутить? Только я не знаю как их лучше реализовать (чтобы динамически подгружалось), сейчас есть поле board_cat в объекте массива с досками.

Могу добавить пару строк, чтобы делать сплит по "|" в массивы

"info" : {
     "category_order" : "Обсуждения|Общее|Японская культура",
     "board_order" : "a|b|d|diy|elf|h|l|mg|mu"
}

Или добавить сортированные массивы

"info" : {
      "category_array" : [
         "Обсуждения",
         "Общее",
         "Японская культура"
      ],
      "board_array" : [
         "a",
         "b",
         "d",
         "diy",
         "elf",
         "h",
         "l",
         "mg",
         "mu"
      ],
}

UPD: Welp, думаю оставлю массивы
UPD: Алсо, добавь капкод мода

adminpost == 1 //capcode - Team
adminpost == 2 //capcode - Mod

Mishiranu · 2016-07-22T16:02:13Z

max_comment_length

Точно максимальное число символов? Не байт?

lisanyan · 2016-07-22T16:13:40Z

Байт, но сейчас исправлю.

Mishiranu · 2016-07-22T16:17:59Z

Печенька auth переименована в wakaauth

А b/api/authorize как работает? Его параметр тоже wakaauth называется или по прежнему auth?

Mishiranu · 2016-07-22T16:25:27Z

Байт, но сейчас исправлю.

Лол, я сейчас в d пытался пост отправить, мне в качестве кода ошибки возвращалась длина строки. Я так и понял, что ты длину строки тестируешь.

lisanyan · 2016-07-22T16:26:42Z

Лол, я сейчас в d пытался пост отправить, мне в качестве кода ошибки возвращалась длина строки. Я так и понял, что ты длину строки тестируешь.

Да, лол.

Done, теперь чекает по символам.

Параметр по прежнему auth называется.

Mishiranu · 2016-07-22T16:32:55Z

Готово всё. Обновлять?

lisanyan · 2016-07-22T16:37:29Z

Йеп.

lisanyan · 2017-04-03T15:11:24Z

Можешь альт. домен обновить?

02ch.info мертв, новый - buttflaps.pp.ua

Mishiranu · 2017-04-04T20:45:43Z

Готово.

lisanyan · 2018-02-08T17:22:46Z

Сменили движок на meguca (на тохочане тоже), соотвественно апи поменялся, можешь запилить?

Описание здесь
Алсо у объекта треда еще поля sticky и locked, но в доке не описаны.

lisanyan closed this as completed Jun 20, 2016

lisanyan closed this as completed Jul 14, 2016

lisanyan reopened this Jul 22, 2016

lisanyan changed the title ~~Fix pomf links #nulldvachin~~ #nulldvachin Jul 22, 2016

Mishiranu added extension enhancement labels Apr 15, 2017

#nulldvachin #1

#nulldvachin #1

Comments

lisanyan commented Jun 20, 2016 • edited Loading

Mishiranu commented Jun 20, 2016

lisanyan commented Jun 20, 2016

Mishiranu commented Jun 20, 2016

lisanyan commented Jun 20, 2016

Mishiranu commented Jun 20, 2016

lisanyan commented Jun 20, 2016

Mishiranu commented Jun 20, 2016

lisanyan commented Jun 20, 2016

Mishiranu commented Jun 20, 2016

lisanyan commented Jun 20, 2016 • edited Loading

lisanyan commented Jun 20, 2016 • edited Loading

Mishiranu commented Jun 20, 2016 • edited Loading

lisanyan commented Jun 20, 2016

Mishiranu commented Jun 20, 2016

lisanyan commented Jun 20, 2016

lisanyan commented Jun 20, 2016

Mishiranu commented Jun 20, 2016

lisanyan commented Jun 20, 2016

Mishiranu commented Jun 20, 2016

lisanyan commented Jun 20, 2016 • edited Loading

Mishiranu commented Jun 20, 2016

Mishiranu commented Jun 20, 2016

Mishiranu commented Jun 20, 2016

lisanyan commented Jun 20, 2016 • edited Loading

lisanyan commented Jun 20, 2016 • edited Loading

lisanyan commented Jun 20, 2016

Mishiranu commented Jun 20, 2016

lisanyan commented Jun 20, 2016

Mishiranu commented Jun 20, 2016

Mishiranu commented Jul 14, 2016

Mishiranu commented Jul 14, 2016

lisanyan commented Jul 14, 2016

Mishiranu commented Jul 14, 2016

lisanyan commented Jul 14, 2016

lisanyan commented Jul 15, 2016

Mishiranu commented Jul 15, 2016

Mishiranu commented Jul 15, 2016

lisanyan commented Jul 15, 2016

lisanyan commented Jul 15, 2016

Mishiranu commented Jul 15, 2016 • edited Loading

Mishiranu commented Jul 15, 2016

lisanyan commented Jul 15, 2016

Mishiranu commented Jul 15, 2016

lisanyan commented Jul 22, 2016 • edited Loading

Mishiranu commented Jul 22, 2016

lisanyan commented Jul 22, 2016

Mishiranu commented Jul 22, 2016 • edited Loading

Mishiranu commented Jul 22, 2016

lisanyan commented Jul 22, 2016 • edited Loading

Mishiranu commented Jul 22, 2016

lisanyan commented Jul 22, 2016

lisanyan commented Apr 3, 2017

Mishiranu commented Apr 4, 2017

lisanyan commented Feb 8, 2018

lisanyan commented Jun 20, 2016 •

edited

Loading

lisanyan commented Jun 20, 2016 •

edited

Loading

lisanyan commented Jun 20, 2016 •

edited

Loading

Mishiranu commented Jun 20, 2016 •

edited

Loading

lisanyan commented Jun 20, 2016 •

edited

Loading

lisanyan commented Jun 20, 2016 •

edited

Loading

lisanyan commented Jun 20, 2016 •

edited

Loading

Mishiranu commented Jul 15, 2016 •

edited

Loading

lisanyan commented Jul 22, 2016 •

edited

Loading

Mishiranu commented Jul 22, 2016 •

edited

Loading

lisanyan commented Jul 22, 2016 •

edited

Loading