rebased and tested

• Contract all the long searches into a scope i.e. obj.hostgroup and obj.hostgroup.puppetclasses and (klasses = obj.hostgroup.puppetclasses.visible_is(true)).is_a?(Array) and obj.is_a?(Host)
• Set the default scope to only show visible classes
• Update the search facility to allow visible to be used as a search target
• When importing classes, all classes should be imported but a facility should be provided so that some can be marked as invisible
• Update puppetdoc construction so that invisible classes are not processed
• Provide a "toggle all" facility on the puppetclasses page. Use this:
  function checkAll (id, checked) {
  $(id).find(":checkbox:not([disabled='disabled'])").attr('checked',checked)
  }
• Provide a multiple interface so that this operation can be performed on large numbers of classes at once.

I pushed the first version of the patch I'm working on to implement this feature.

1. The smart-proxy is modified to read modelines in Puppet classes' documentation. At this stage, the modeline content isn't parsed at all: only the "foreman:" prefix is read by the smart-proxy. The text following the prefix is then transfered as-is to Foreman.
2. Foreman is modified to parse the modelines sent by the smart-proxy and store it in the database. A modeline is a one-line JSON hash. Later, when editing a host or a hostgroup for instance, the modeline is used to determine if a Puppet class should be displayed. A new button allows one to toggle the display of internal classes.

Here's an example of a modeline in a Puppet class:

# Class: mymodule::myclass
#
# foreman: { internal-class }
class mymodule::myclass {
    ...
}

• https://github.com/yakaz/smart-proxy/tree/modeline
• https://github.com/yakaz/foreman/tree/fix418-hide-internal-puppet-classes

Caveat:
Both branches are based on Olivier Favre's branches regarding Parameterized classes, because I needed the modifications to the protocol between the smart-proxy and Foreman.

I wanted my foreman to only display my roles and profiles classes only. I achieved this by creating this file:

$ cat /usr/share/foreman/config/ignored_environments.yml
#:ignored:
#  - test

:filters:
  - !ruby/regexp '/^(?!role|profile).*$/'

This didn't work at first but then I realized that it does work as long as you create this file before doing any foreman class/environment imports.

Hope this helps for anyone else having issues using the ignored_environments.yml file.