Bnei-Baruch/mdb

Method AssociateByLikutim.createCollection has 7 return statements (exceeds 4 allowed).

Open

func (a *AssociateByLikutim) createCollection(uid string) (*models.Collection, error) {
    startDate, err := a.findStartDate(uid)
    if err != nil {
        return nil, err
    }

Method AssociateBySources.createCollection has 6 return statements (exceeds 4 allowed).

Open

func (a *AssociateBySources) createCollection(uid string) (*models.Collection, error) {
    startDate, err := a.findStartDate(uid)
    if err != nil {
        return nil, err
    }

Function attachCollection has 5 return statements (exceeds 4 allowed).

Open

func attachCollection(tx boil.Executor, c *models.Collection, cu *models.ContentUnit, cus []int64) error {
    var prevPos int64

    if err := queries.Raw(fmt.Sprintf(LAST_POSITION_BY_C_SQL, c.ID)).QueryRow(tx).Scan(&prevPos); err != nil {
        return err

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    for sUid, _ := range a.seriesSources {
        if len(a.cusByS[sUid]) < MinCuNumberForNewLessonSeries {
            continue
        }
        c, err := a.findPrevCollection(sUid)

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    for _, lUid := range a.lUIDs {
        if len(a.cusByL[lUid]) < MinCuNumberForNewLessonSeries {
            continue
        }
        c, err := a.findPrevCollection(lUid)

• Read up
  Read up