[PATCH/foreman 1/1] fixes #1299 - foreman don't remove the () from puppet class names

Development
1

Signed-off-by: Florian Koch <florian.koch1981@googlemail.com>

··· From: Florian Koch --- app/models/puppetclass.rb | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/app/models/puppetclass.rb b/app/models/puppetclass.rb
index 9801935…58c4157 100644
— a/app/models/puppetclass.rb
+++ b/app/models/puppetclass.rb
@@ -38,7 +38,7 @@ class Puppetclass < ActiveRecord::Base
for path in paths.split(":")
Dir.glob("#{path}//manifests/**/.pp").each do |manifest|
File.read(manifest).each_line do |line|

  •      klass=line.match(/^class (\S+).*\{/)

  • klass=line.match(/^class (\S+)(\(\)*)/)
     klasses << klass[1] if klass
   end
 end

–
1.7.6.4

2

STOP!! the regex is not the right

··· On Nov 7, 11:52 am, Florian Koch wrote: > From: Florian Koch > > Signed-off-by: Florian Koch > --- > app/models/puppetclass.rb | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/app/models/puppetclass.rb b/app/models/puppetclass.rb > index 9801935..58c4157 100644 > --- a/app/models/puppetclass.rb > +++ b/app/models/puppetclass.rb > @@ -38,7 +38,7 @@ class Puppetclass < ActiveRecord::Base > for path in paths.split(":") > Dir.glob("#{path}/*/manifests/**/*.pp").each do |manifest| > File.read(manifest).each_line do |line| > - klass=line.match(/^class (\S+).*\{/) > + klass=line.match(/^class (\S+)(\(\)*)/) > klasses < end > end > -- > 1.7.6.4