fixing portfolio class