Utiliser les helpers dans rails 3, à la sortie html

J'essaye de faire de mon mieux pour construire un programme d'assistance que les sorties a <'ul> composé de tous les membres d'une collection. Pour chaque membre de la collection je veux imprimer un < li> qui a un titre, et une div de liens vers CRUD le membre. C'est assez similaire à ce que les Rails de sorties pour l'échafaudage pour la vue index.

Voici l'aide que j'ai eu:

def display_all(collection_sym)
  collection = collection_sym.to_s.capitalize.singularize.constantize.all

  name = collection_sym.to_s.downcase

  html = '' 

  html << "<ul class=\"#{name}-list\">"

  for member in collection do
    html << content_tag(:li, :id => member.title.gsub(' ', '-').downcase.strip) do
     concat content_tag(:h1, member.title, :class => "#{name}-title")
     concat link_to 'Edit', "/#{name}/#{member.id}/edit"
     concat "\|"
     concat link_to 'View', "/#{name}/#{member.id}"
     concat "\|"
     concat button_to 'Delete', "/#{name}/#{member.id}", :confirm => 'Are you sure?  This cannot be undone.', :method => :delete
    end
   end

   html << '</ul>'

 return html
end

Et que la production exactement ce que je veux. Tout d'abord, si quelqu'un pense qu'il y a une meilleure façon de le faire, n'hésitez pas à me corriger, je soupçonne que je suis en train de faire cela dans un bass ackwards façon, mais pour le moment c'est le seul moyen que je sais faire.

J'ai ensuite essayé d'envelopper les liens dans un div comme suit:

def display_all(collection_sym)
  collection = collection_sym.to_s.capitalize.singularize.constantize.all

  name = collection_sym.to_s.downcase

  html = '' 

  html << "<ul class=\"#{name}-list\">"

  for member in collection do
     html << content_tag(:li, :id => member.title.gsub(' ', '-').downcase.strip) do
     concat content_tag(:h1, member.title, :class => "#{name}-title")
     concat content_tag(:div, :class => "links-bar") do
       concat link_to 'Edit', "/#{name}/#{member.id}/edit"
       concat "\|"
       concat link_to 'View', "/#{name}/#{member.id}"
       concat "\|"
       concat button_to 'Delete', "/#{name}/#{member.id}", :confirm => 'Are you sure?  This cannot be undone.', :method => :delete
     end
   end
 end

 html << '</ul>'

 return html
end

Cependant, j'ai maintenant plus d'obtenir toutes les balises à l'intérieur de la div.liens-bar sortie à la vue. Je suis sûr que cela doit avoir quelque chose à voir avec le bloc et les liaisons, mais je peux pas pour la vie de me comprendre à quoi ou comment le corriger. Quelqu'un peut-il offrir une aide?

Quelle est votre première intention utiliser les helpers? Pourquoi ne pas faire cela dans le modèle, à la place?
Hmmmm, je suppose que je pourrais utiliser un modèle, je ne sais pas pourquoi je n'avais pas pensé à ça.
les partiels est le chemin à parcourir sur ce que je pense...bravo pour labourer par le biais de ce code...mais j'ai un mal de tête peu de le lire 😉
Le problème de base n'est pas des blocs ou des liaisons, mais que la chaîne "html", vous êtes en train de créer est marquée comme non-HTML-fort. Vous pouvez utiliser les premières (le) de la fonction, même si, comme d'autres l'ont dit, partiels ou content_tag sont bien meilleures idées. Juste pensé que je ferais remarquer que le problème sous-jacent est, pour les gens qui sont similaires, mais différentes, des problèmes plus tard.

InformationsquelleAutor TheDelChop | 2010-08-24

46

Je suis d'accord avec le commentaire ci-dessus recommandent l'utilisation d'un partiel... mais si vous AVEZ besoin de faire un assistant, ce qui est une façon plus propre à mettre en œuvre:
```
def display_all(collection)
  content_tag(:ul, class: "list") do
    collection.collect do |member|
      concat(content_tag(:li, id: member.name.gsub(' ', '-').downcase.strip) do
        member.name
      end)
    end
  end
end
```
J'avais passer dans une collection explicitement plutôt que de passer par un symbole pour créer une collection de sorte que vous ne sont pas toujours requis pour l'affichage de TOUS les enregistrements dans une table particulière à la fois. Vous pouvez ajouter la pagination, etc.
- Ouais je suis d'accord que si je devais le faire de cette façon, c'est la meilleure façon de le faire. Mais comme l'autre gars a dit, je vais utiliser un partiel à la place. Merci!
- n'avez-vous pas besoin d'un concat pour l'intérieur content_tag?
InformationsquelleAutor Dave Pirotte
24

@Joe,
Vous pouvez toujours utiliser votre méthode display_all(collection_sym)
Utilisez simplement:
return html.html_safe
au lieu de:
return html

Que je trouve encore dans de nombreuses situations, il est préférable de générer du HTML à partir d'aides, au lieu d'utiliser des partiels. Ainsi, le html_safe fonction dans Rails 3 sera assurez-vous de générer du HTML au lieu de le convertir à String.
- A travaillé comme un charme pour moi, et qui est agréable et simple lorsque j'ai seulement besoin de générer un point ou deux de l'aide.
InformationsquelleAutor Linh Chau

Comme @TheDelChop dit, vous avez besoin d'un concat pour l'intérieur content_tag, sinon le résultat est juste <ul></ul>

Voici à quoi ça ressemble:

def display_all(collection)
  content_tag(:ul, :class => "list") do
    collection.collect do |member|
      concat(
        content_tag(:li, :id => member.name.gsub(' ', '-').downcase.strip) do
          member.name
        end
      )
    end
  end
end

Plus d'explication ici: L'imbrication content_tag dans Rails 3

InformationsquelleAutor James Hibbard

Vous devez vous connecter pour publier un commentaire.