LINUX.ORG.RU
ФорумTalks

Про LOR-API и реквесты которые никто не принимает.

 ,


0

1

Вот я взял и посмотрел на «невинно не принятый» пулл:

https://github.com/maxcom/lorsource/pull/642/files

From 03b8cfd31484bb95cfb600a77755b7794c928694 Mon Sep 17 00:00:00 2001
From: 
Date: Tue, 2 Jun 2015 12:39:50 +0300
Subject: [PATCH] Add "/api" prefixes

---
 src/main/java/ru/org/linux/user/UserEventApiController.java | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/main/java/ru/org/linux/user/UserEventApiController.java b/src/main/java/ru/org/linux/user/UserEventApiController.java
index a15c68e..a9291cf 100644
--- a/src/main/java/ru/org/linux/user/UserEventApiController.java
+++ b/src/main/java/ru/org/linux/user/UserEventApiController.java
@@ -35,7 +35,7 @@
   private UserEventService userEventService;
 
   @ResponseBody
-  @RequestMapping(value = "/notifications-count", method= RequestMethod.GET)
+  @RequestMapping(value = "/api/notifications-count", method= RequestMethod.GET)
   public int getEventsCount(HttpServletRequest request, HttpServletResponse response) throws Exception {
     Template tmpl = Template.getTemplate(request);
     if (!tmpl.isSessionAuthorized()) {
@@ -47,7 +47,7 @@ public int getEventsCount(HttpServletRequest request, HttpServletResponse respon
     return tmpl.getCurrentUser().getUnreadEvents();
   }
 
-  @RequestMapping(value="/notifications-reset", method = RequestMethod.POST)
+  @RequestMapping(value="/api/notifications-reset", method = RequestMethod.POST)
   @ResponseBody
   public String resetNotifications(
     HttpServletRequest request,
@@ -66,7 +66,7 @@ public String resetNotifications(
   }
 
   @ResponseBody
-  @RequestMapping(value = "/yandex-tableau", method = RequestMethod.GET, produces={"application/json"})
+  @RequestMapping(value = "/api/yandex-tableau", method = RequestMethod.GET, produces={"application/json"})
   public Map<String, Integer> getYandexWidget(HttpServletRequest request, HttpServletResponse response) throws Exception {
     Template tmpl = Template.getTemplate(request);
     if (!tmpl.isSessionAuthorized()) {
@@ -78,7 +78,7 @@ public String resetNotifications(
     }
   }
 
-  @RequestMapping(value = "/notifications-list", method = RequestMethod.GET, produces = "application/json; charset=UTF-8")
+  @RequestMapping(value = "/api/notifications-list", method = RequestMethod.GET, produces = "application/json; charset=UTF-8")
   @ResponseBody
   public Map<String, Object> listNotifications(
           @RequestParam(value = "filter", required = false) String filterAction,

Для тех кто не понял, скажу, коммит ломает обратную совместимость с имеющимися кодом, причем «просто так».

Сломано просто так, блиан! На адекватное работе за такое эльфийское поведение делают предупредительную прочистку мозгов, если не помогло то увольняют.

С подобным я встречался в своей практике, только тогда мне бедняжко не смогло мотивировать свое поведение, может тут на лоре есть кто-то кто просветит?

Deleted

Ответ на: комментарий от Deleted

С чего ты взял, что человек, пожелавший сделать для вас АПИ безвозмездно не выслушает твои замечания? Что за нытьё попусту?

Я не понимаю откуда столько злобы.

Deleted
()
Последнее исправление: Deleted (всего исправлений: 1)
Ответ на: комментарий от i-rinat

Пока эта ветка у тебя на локальной машине или в личном репозитории, поступай как хочешь.

По ссылке и есть личный репозиторий.

Ветка master — и есть стабильная ветка.

Там об этом ни слова. Т.к. там нет разделения на stable и devel, очевидно что master это devel.

Flame4All
()
Ответ на: комментарий от Flame4All

А, так его ещё не приняли. Выфер, ты всё та же стервозная истеричка.

Слышь ты эта не проецируйся на меня 8)

Deleted
()
Ответ на: комментарий от stevejobs

нету никакой стабильной ветки

Тем более нет причин для батхёрта.

Flame4All
()
Ответ на: комментарий от stevejobs

именно, хотя с виду макс публично вроде ничего не ответил, но авторы пуллов писали что макс не принимает, короче на лицо недосказанность, ну я полез посмотреть что там за волшебные коммиты такие

Deleted
()

с имеющимися кодом

Такой есть? Иногда лучше поломать совместимость ради логики.

Klymedy ★★★★★
()

коммит ломает обратную совместимость с имеющимися кодом

А что, LOR API уже дошел до стабильного релиза?

tailgunner ★★★★★
()
Ответ на: комментарий от Deleted

моим подчиненным

Дебилам, которые хардкодят данные в код, еще и подчиненных выдают? О времена, о нравы.

Deleted
()
Ответ на: комментарий от Klymedy

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

Deleted
()
Ответ на: комментарий от Deleted

А что, LOR API уже дошел до стабильного релиза?

F12 нажми и увидишь, чудо

О чем ты, болезный? Там будет написано «LOR API стабилен»?

tailgunner ★★★★★
()
Последнее исправление: tailgunner (всего исправлений: 1)
Ответ на: комментарий от Deleted

Тебе везде поциенты из твоего привычного окружения мерещатся?

Почему во множественном числе? Поциент здесь только один - ты.

tailgunner ★★★★★
()
Ответ на: комментарий от tailgunner

Потому что у тебя все поциенты, и каждому ты говоришь о его уникальности 8) Вообще иногда ты себя так ведешь что твой ник переводится как «стреляющий из под хвоста», видимо пламенем, да.

Deleted
()
Ответ на: комментарий от Tanger

Там в каждом реквесте есть такие коммиты, даже в принятых, макс даже в коментах к реквесту както просил почистить.

Deleted
()
Ответ на: комментарий от Deleted

Потому что у тебя все поциенты

«Слышь ты эта не проецируйся на меня 8)» (ц)

и каждому ты говоришь о его уникальности 8)

То, что ты здесь один такой болезный, не значит, что ты уникален.

tailgunner ★★★★★
()
Ответ на: комментарий от Deleted

макс даже в коментах к реквесту както просил почистить.

Так дискасс пулреквестов - это нормально.

Там в каждом реквесте есть такие коммиты

Но да, это грустно.

Tanger ★★★★★
()

1. maxcom выразил согласие на приведение URL API к единообразному виду здесь: https://github.com/maxcom/lorsource/pull/640#discussion_r31465892 Стоит отметить, что ответа на PR из шапки не последовало, поэтому неизвестно мнение maxcom по поводу совместимости в данном случае. 2. Лес рубят - щепки летят. Отправить PR может любой желающий, и если вы можете предложить свое решение, как лучше сделать API, то, уверен, заинтересованные в написании приложений/скриптов для ЛОРа будут только рады.

За сим откланяюсь.

mcgeek
()
Ответ на: комментарий от mcgeek

maxcom выразил согласие на приведение URL API к единообразному виду здесь

я это видел, только вот додуматся заниматься впиливанием мелкой функции и захирячить тудаже рефакторинг (а не отдельно) это надо додуматься же

Deleted
()
Вы не можете добавлять комментарии в эту тему. Тема перемещена в архив.