Widgets are broken about fortitude HOT 12 CLOSED

Note that the problem goes away if I either a) use the render partial: "form" syntax, or b) fully qualify the widget name widget(Views::Foo::Form, bar: bar). I'm OK living with that, but maybe some documentation would help?

from fortitude.

Whoops — definitely an oversight. Yeah, the triggering case was referring to an autoloaded widget by partially-qualified module path, and I actually never had a test case that did that. Now I do. 😄

(When you write widget(Form, … above, inside Views::Foo::Edit, Rails very first looks for something called Views::Foo::Edit::Form due to namespacing conventions. My code was hence looking for a file named app/views/foo/edit/form.(rb|html.rb) by scanning the directory app/views/foo/edit; it correctly dealt with the case where form.rb didn't exist, but not where app/views/foo/edit didn't. Now it does.)

Let me know if this works for you — if so, I'll cut 0.0.7 based on it.

from fortitude.

It works... mostly. The widget line no longer throws errors. But, if I change and reload the Views::Foo::Edit file too many times, eventually I will get this error:

ActionView::Template::Error (superclass mismatch for class Edit):
    1: module Views
    2:   module Foo
    3:     class Edit < Views::Base

I have seen the 'superclass mismatch' errors before, so this may be a separate issue.

from fortitude.

Oh, and I've been able to avoid the 'superclass mismatch' problem in the past by adding the following line to the top of every view file:

require_dependency 'views/base'

Just tried it, and it seems to work in this case too.

from fortitude.

Hmmm. I'll take a look and see what I can find — I haven't personally run into this problem, but I'll see if I can reproduce it.

If you find a way to reproduce it even semi-reliably, definitely let me know; I am obviously extremely interested in stamping such things out. :)

from fortitude.

I just pushed a branch, ageweke/classloading_debugging, that contains a single fix that I think might help — it adds a require_dependency of the widget file itself into the generated template code. If you continue having issues, you might try this branch and see if it helps at all.

from fortitude.

Hmm, don't see the branch in your repo. Where is it pushed to?

from fortitude.

Oops — sorry about that. Fixed now. :)

from fortitude.

That branch doesn't really change things for me.

I've been able to consistently reproduce the error. Would you prefer that info in a new issue or should I put it here?

from fortitude.

Sorry to hear that! By all means, if you can consistently reproduce it, file a new issue. Your bug report above was excellent in terms of thoroughness — that level of detail should be able to help me figure it out. The output of ruby --version and rails --version would be awesome, too.

from fortitude.

Oh, and BTW, sorry for the delay. I'm quite scrupulous about not working on OSS during my employer's time or while on their property, so during daytime business hours (PST), I'm unlikely to respond.

from fortitude.

Totally understand... and you're fixing things pretty fast anyway. Thanks again!

from fortitude.