Welp Sean has pointed out that this has been fixed in edge Radiant and all appears hunky-dory.

The change is a one-liner is here - http://github.com/radiant/radiant/blob/master/app/models/standard_tags.rb#L980


Jeff

Jeffrey Jones wrote:
Yup, also getting this error 0.7.1. Bit of a showstopper as far as the comments extension goes.

A - vaguely works- workaround is to take the user back to the page the comment was posted from. They won't see their comment or get any hint that it is awaiting approval but at least the site doesn't explode.

 def create
   comment = @page.comments.build(params[:comment])
   comment.request = request
   comment.save!

   ResponseCache.instance.clear
CommentMailer.deliver_comment_notification(comment) if Radiant::Config['comments.notification'] == "true"

   flash[:selected_comment] = comment.id
   redirect_to "#...@page.url}#comment-#{comment.id}" <---

Doesn't help if they fail validation though.

Jeff


Victor Elsendoorn wrote:
Thanks Simon, for your reply. However this doesn't solve the problem. The error remains:

    The error occurred while evaluating nil.relative_url_root.

and by the way the radiant version I'm using is indeed 0.7.1

Other ideas?

Thanks in advance.




On Apr 2, 2009, at 5:24 PM, Simon Josi wrote:

I'm still using 0.6.9, I don't know if this issue is fixed for 0.7.x.
I've fixed this in my fork of the comments extensions:
http://github.com/turingmachine/radiant-comments/commit/c5efa910e5c83ebd7402a7c263064c6f28741bd1

/simon

On Thu, 2 Apr 2009 17:01:27 +0200
Victor Elsendoorn <elsendo...@w3craft.nl> wrote:

Dear list,

I found out having trouble with the comments extension in
combination with the usages of a navigation snippet based on using
the <r:link /> tag.

It seems that it has something to do with the fact that the
<r:link /> tag is producing relative url's rather than absolute
url's. Can someone tell me how to use absolute url's instead of the
relative ones?

By the way, the error produced by the comments controller: not
expecting nil_relative_url_for

Hope someone can help me out here.

Regards,
Victor
_______________________________________________
Radiant mailing list
Post:   Radiant@radiantcms.org
Search: http://radiantcms.org/mailing-list/search/
Site:   http://lists.radiantcms.org/mailman/listinfo/radiant
_______________________________________________
Radiant mailing list
Post:   Radiant@radiantcms.org
Search: http://radiantcms.org/mailing-list/search/
Site:   http://lists.radiantcms.org/mailman/listinfo/radiant

_______________________________________________
Radiant mailing list
Post:   Radiant@radiantcms.org
Search: http://radiantcms.org/mailing-list/search/
Site:   http://lists.radiantcms.org/mailman/listinfo/radiant
_______________________________________________
Radiant mailing list
Post:   Radiant@radiantcms.org
Search: http://radiantcms.org/mailing-list/search/
Site:   http://lists.radiantcms.org/mailman/listinfo/radiant
_______________________________________________
Radiant mailing list
Post:   Radiant@radiantcms.org
Search: http://radiantcms.org/mailing-list/search/
Site:   http://lists.radiantcms.org/mailman/listinfo/radiant

Reply via email to