Made a ticket (222) and added the patch. Regards, Jeroen
Jeroen van Doorn schreef: > Hi James, > > No, I didn't. Submitting patches and stuff is new for me, but I will do > that today. > > Regards, > Jeroen > > James Adam schreef: > >> Hi Jeroen, >> >> Did you submit this as a ticket? http://dev.rails-engines.org >> >> Cheers, >> >> James >> >> On 9/15/06, Jeroen van Doorn <[EMAIL PROTECTED]> wrote: >> >> >>> Hi all, >>> >>> I think I've located some superfluous code. I just can't see what it >>> does, so I've created a patch. Please tell me if this code has a >>> function somehow. >>> >>> Kind regards, >>> Jeroen van Doorn >>> >>> >>> Index: role_controller.rb >>> =================================================================== >>> --- role_controller.rb (revision 453) >>> +++ role_controller.rb (working copy) >>> @@ -106,9 +106,7 @@ >>> @role.permissions.clear >>> >>> permissions.each { |perm| >>> - if [EMAIL PROTECTED](perm) >>> @role.permissions << perm >>> - end >>> } >>> >>> # save the object >>> >>> >>> _______________________________________________ >>> engine-users mailing list >>> [email protected] >>> http://lists.rails-engines.org/listinfo.cgi/engine-users-rails-engines.org >>> >>> >>> >>> >>> >> >> > > _______________________________________________ > engine-users mailing list > [email protected] > http://lists.rails-engines.org/listinfo.cgi/engine-users-rails-engines.org > > > > _______________________________________________ engine-users mailing list [email protected] http://lists.rails-engines.org/listinfo.cgi/engine-users-rails-engines.org
