Same comments I had for fabiano, generally in rails you don't abstract model fetch like this.  Instead duplicate it in each method so it's always crystal clear what instance variables are available.  Only abstract when it's more than a few lines, when the logic is suspect, etc.