Project

General

Profile

Actions

Bug #6213

closed

Builder::XmlMarkup drops html_safe

Added by eike.rb (Eike Dierks) about 12 years ago. Updated about 12 years ago.

Status:
Third Party's Issue
Assignee:
-
Target version:
-
ruby -v:
1.9.2
Backport:
[ruby-core:<unknown>]

Description

A single line in Builder::XmlMarkup needs to be changed slightly,
to enable the builder to work with the html_safe flag that was introduced with rails3.

like

def method_missing(sym, *args, &block)
      [...]
      text = arg.to_s   # changed this line to keep the html_safe flag

Updated by nobu (Nobuyoshi Nakada) about 12 years ago

  • Status changed from Open to Feedback
  • Priority changed from 5 to 3

What is Builder::XmlMarkup?

Updated by shyouhei (Shyouhei Urabe) about 12 years ago

  • Status changed from Feedback to Third Party's Issue

I guess it is the builder gem.

Updated by eike.rb (Eike Dierks) about 12 years ago

shyouhei (Shyouhei Urabe) wrote:

I guess it is the builder gem.

Hi Shyouhei,
you're probably right that this is in a third party gem,
but the xml builder is quite a central gem.
so you might want to give me some advice,
to whom I'd should report this?

Thinking about that,
I'd really want to get the builder into the core of ruby.
So that we can improve on the builder within here.

I'd like to improve the builder, so that it would work nicely
with the html_safe of rails3

Which actually boils down to not escaping things,
that do have the safe flag set.

And declaring all output of the xmlbuilder html_safe
by definition.

It was not really easy to inject into the current xmlbuilder.
And it's not yet really safe.

I want to get the builder into the core of ruby (and rails)
and to make it a first line aproach for building markup.

And I want to make it really safe.

I had a close look at the code base of the builder,
(it works quite well as of today)

But we might need to rewrite that.
But the builder concept is big!

Updated by naruse (Yui NARUSE) about 12 years ago

eike.rb (Eike Dierks) wrote:

you're probably right that this is in a third party gem,
but the xml builder is quite a central gem.
so you might want to give me some advice,
to whom I'd should report this?

Why don't you google it?

Updated by nobu (Nobuyoshi Nakada) about 12 years ago

=begin
"metadata" in builder-3.0.0.gem contains:
authors:

Updated by eike.rb (Eike Dierks) about 12 years ago

Hi nobu,
I wrote to jim

I want to make this builder concept
into the core of ruby.

I just love this idea,
of an object of being in full controll
of all and any messages send to it.

That's a proxy

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0Like0