Are they doing the right thing? What has been added to the beansession api that makes some of this code unnecessary? A few paragraphs proposing changes or saying it all makes sense are probably enough.