Feature/805 project structure

What is the purpose of this line? Does it not simply return the same item?

```
item = super.getByKey(item.getKey().getId());
```

On Tue, Nov 18, 2014 at 7:09 PM, Iván Perdomo <notifications@github.com>
wrote:

> In GAE/src/com/gallatinsystems/survey/dao/SurveyGroupDAO.java:
>
> >       * @param item
> >       */
> >      public void delete(SurveyGroup item) {
> > -        // This probably won't work on the server
> > -        SurveyDAO surveyDao = new SurveyDAO();
> > -        item = super.getByKey(item.getKey().getId());
> > -        for (Survey survey : surveyDao
> > -                .listSurveysByGroup(item.getKey().getId())) {
> > -            SurveyTaskUtil.spawnDeleteTask("deleteSurvey", survey.getKey()
> > -                    .getId());
> > -        }
> > -        super.delete(item);
> > +	// This probably won't work on the server
> > +	SurveyDAO surveyDao = new SurveyDAO();
> > +	item = super.getByKey(item.getKey().getId());
>
> We shouldn't reassign the value of item. Let's do it by creating a local
> variable.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/akvo/akvo-flow/pull/916/files#r20520535>.
>