да

а потом делаешь if !chapter.new_record? render json: chapter else render json: [book.errros, chapter.errors] end

как-то так

в случае тенпентая надо if books.chapters.size < 5 if chapter.save render else render end else render end

это зависит от степени твоей извращенности

я выше предложил вариант попроще

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

у меня вообще чаптер сохранялся chapter.save

в принципе решение норм поставить на class Chapter belongs_to :book, autosave: true end

извращения, извращения

тык наоборот же проще добавить autosave: true чем эта елка из ифов

class Chapter belongs_to :book, counter_cache: true validate :deny_extra_chapters def deny_extra_chapters errors.add(:base, 'В книге уже есть 5 глав') if book.chapters_count == 5 end end

елку из ифов ты сам выдумал

в Book добавь поле chapters_count

как-то так

class Chapter belongs_to :book, counter_cache: true validate :deny_extra_chapters def deny_extra_chapters errors.add(:base, 'В книге уже есть 5 глав') if book.chapters_count == 5 end end

вот нафига в чаптеры добавлять логику проверки их количества?

ооокей, как без елки из ифов?

ооокей, как без елки из ифов?

if !chapter.new_record? render json: chapter else render json: [book.errros, chapter.errors] end

а до этого нет ифов?

до этого нет

Что бы ошибка была на чаптере, но мне не очень нравится такой подход)

КАК???

Что бы ошибка была на чаптере, но мне не очень нравится такой подход)

зачем этой ошибке быть на чаптере?

перебор глав - это свойство книги

КАК???

save

а

Это вопрос не ко мне)) Это было изначальное условие задачи)) Я согласен с тем что проверку кол-ва чаптеров надо делать на буке

ну один иф будет

def create @chapter = @book.chapters.new(chapter_params) if @book.save render json: @chapter, include: 'link' else render json: @chapter.errors, status: :unprocessable_entity end end

чот жесть

> @silentshade Я так понимаю проблема такая: Надо НЕ сохранять чаптер если он вылезает за рамки 5 у бука, так? При этом ошибку хочется видеть на чаптере а не на буке.. Тут в любом случае надо запрашивать сколько чаптеров у бука, иначе откуда чаптер узнает что он уже 6 > @vasivas да

ну с autosave проверка остаесть на буке

ты зачем книгу сохраняешь?

Я так понимаю чаптеры сохраняются отдельно

от книг

и вот

если это уже 6ой

то его не надо сохранять

если я правильно все понял

потому что @chapter.save возратит true ну

потому что @chapter.save возратит true ну

ле, сделай сохранеиние по условию

считай, не перебор ли глав и сохраняй, если не перебор

это то же самое, только ты предлагаешь считать в контроллере, а я в валидации

в целом в контроллере наверное правильней, но в валидации можно получить нативную ошибку в модели..

кароч autosave: true решает всё

на belongs_to :city

что он решает?

проблему проверки количества дочерних записей?

сомневаюсь

@chapter.save вернет false если глав больше

А валидация на книге стоит?

как?

без валидации?

autosave тебе просто сохраняет связанные модели ДО сохранения главной в одной транзакции

на книге да

как вы и советуете, лол)

@chapter.errors что говорит?

после сохранения?

и @book.errors

@chapter.erros говорит {"book.chapters":["слишком большой длины (не может быть больше чем 5 символов)"]} (сообщение в данном случае не очень имеет значение)

а почему оно в chapter.errors, а не в book.errors?

def create @chapter = @book.chapters.new(chapter_params) if @book.chapters == 5 render json: :errors, status: :unprocessable_entity else if @chapter.save render json: @chapter else render json: @chapter.errors, status: :unprocessable_entity end end end охуеть же

а зчем первое условие если у тебя уже есть в модели валидация на это?)

хотя да, можно так же лол) def create @chapter = @book.chapters.new(chapter_params) @book.save if !chapter.new_record? render json: @chapter, include: 'link' else render json: @chapter.errors, status: :unprocessable_entity end end

хм, интересно

ну и да, в елсе ошибки с бука подтянуть ещё

ERROR: S client not available

а можешь показать как у тебя валидация написана?

в кол-во

просто я такого не встречал что бы автосейв копировал ошибку

да как-то так validates :chapters, length: { maximum: 5 }

я не первый раз вижу когда parent.child: 'error' идёт вид ошибки

нестед_аттрибутес так делает, т.к. там аутосейв врубается для связей

забавно.. а у меня ошибка только на паренте

а рельса какая?

4.2 но и в 5 так было

я тут кстати за енумы топил в моделях недавно - кароч гавно это, кек)

блин вроде все прописано а не работает) 4.2.7 рельса

ну т.е. если устанавливает не юзер то может вполне и ок, а если юзер, то при кривом значении енум просто нахер летит ексепшен ArgumentError и всё, никакой норм обработки

Энум на то и энум что его не надо давать устанавливать, его можно только выбирать

Скажите, а такая штука в роуте это кака или я привередничаю?

get 'add_like' => 'likes#create', as: :add_like

Создание сущности через get запрос это же не тру или можно?

не REST

заклюют тебя за такое

Да это не я, это наследство. Размышляю переделать ли. Переделаю.

да вообще размножение get/post/match в routes.rb - Это ниочинь

Как можно сократить строку посередине? К примеру есть юрл "https://en.wikibooks.org/wiki/Ruby_Programming/Syntax/Literals" мне надо получить => "https://en.wik...ntax/Literals"

взять энное количество символов с начала и конца строки, в середину впиндюрить многоточие

"#{url[0..10]}....#{url[-10..-1]}"

с короткой строкой заглючит, наеврное

truncate метод есть, там omission параметр вместо многоточий конец строки передавать можно

"#{url[0..10]}....#{url[-10..-1]}"

"#{url[0..10]}....#{url[-10..-1]}"

да, так проще

да и зеленое приятнее

"#{url[0..10]}....#{url[-10..-1]}"

еще вариант, рельсовый "#{url.first(10)}.....#{url.last(10)}"

truncate конец же только режет

url.sub(/\A(.{10}).+(.{10})\Z/m, '\1...\2')

truncate(str, length: 10, omission: "...#{str[-5..-1]}") Если str строка короткая, то обрезаний не будет: str = 'Qwerty' => 'Qwerty'