Rails reorder isn't working? -

-

i have feedback model 2 other models (comment , review) inherit from.

i have default scope in feedback model orders id asc

i've added scope feedback model:

class feedback < activerecord::base   include paginatable    belongs_to :video    default_scope     order(:id => :asc)   end    scope :newest, -> { reorder(:created_at => :desc) }  end

i have video model has:

has_many :feedback  has_many :latest_feedback, -> { newest }, class_name: 'feedback'

the problem i'm having when call

video.includes(:latest_feedback).find(1)

it still orders feedback id first , created_at not want. want latest_feedback ordered created_at

here's does:

video load (0.3ms)  select  `videos`.* `videos`  `videos`.`id` = 85  order `videos`.`id` desc limit 1 feedback load (0.6ms)  select `feedback`.* `feedback`  `feedback`.`video_id` in (1)  order `feedback`.`id` asc, `feedback`.`created_at` desc

here's need do:

video load (0.3ms)  select  `videos`.* `videos`  `videos`.`id` = 85  order `videos`.`id` desc limit 1 feedback load (0.6ms)  select `feedback`.* `feedback`  `feedback`.`video_id` in (1)  order `feedback`.`created_at` desc

update

i guess known issue rails. there still open pull request fix here:

https://github.com/rails/rails/pull/16531

it doesn't land until rails 5.0.0 though.

you can use video.joins(:latest_feedback).find(1). seems rails appends default scope without scope recalculation on includes


Comments

Post a Comment

Popular posts from this blog

java - Custom OutputStreamAppender not run: LOGBACK: No context given for <MYAPPENDER> -

-
i wish write custom appender on basis of outputstreamappender . wrote following class package tests; import java.io.ioexception; import java.io.outputstream; import ch.qos.logback.core.outputstreamappender; public class myappender<e> extends outputstreamappender<e> { public myappender() { system.out.println("myappender created"); setoutputstream(new outputstream() { @override public void write(int b) throws ioexception { throw new unsupportedoperationexception(); } }); } } then wrote following runner: package tests; import org.slf4j.loggerfactory; import ch.qos.logback.classic.logger; public class runner { private static final logger log = (logger) loggerfactory.getlogger(runner.class); public static void main(string[] args) { //log.addappender(new myappender<iloggingevent>()); log.info("hello world"); } } finall...
Read more

java - UML - How would you draw a try catch in a sequence diagram? -

-
Image
is there standard way of drawing try catch sequence diagrams? i've made attempt based on this feel end result doesn't feel right. code diagram based around: public static void save () { try { filehandle filehandle = gdx.files.external(file); filehandle.writestring(boolean.tostring(constantshandler.soundenabled)+"\n", false); (int = 0; < 5; i++) { filehandle.writestring(integer.tostring(constantshandler.highscores[i])+"\n", true); } } catch (throwable e) { } } my attempt note: know still need add loop. that's ok. fragments meant show conditional control flow in sequence diagram. superstructures puts limitations on use of fragments (see pp. 467 of ss2.1.1). should use critical region try part above. , option catch . however, keep telling can take quite freedom in using uml. it's language , such changes in life time. long reader ge...
Read more

c++ - No viable overloaded operator for references a map -

-
i trying use map can make tag names reference number. when try , use it, in code error (6 in total every time reference map): src/main.cpp:25:45: error: no viable overloaded operator[] type 'std::map<std::string, std::string>' const char* idcs = node.child_value(tagmap[3]); this code: #include "pugi/pugixml.hpp" #include <iostream> #include <string> #include <map> int main() { pugi::xml_document doca, docb; std::map<std::string, pugi::xml_node> mapa, mapb; std::map<std::string, std::string> tagmap {{"1", "data"}, {"2", "entry"}, {"3", "id"}, {"4", "content"}}; if (!doca.load_file("a.xml") || !docb.load_file("b.xml")) { std::cout << "can't find input files"; return 1; } (auto& node: doca.child(tagmap[1]).children(tagmap[2])) { co...
Read more