lp://staging/~philsf/mmrrsim/bp-classes-overhaul

Branch merges

Propose for merging

No branches dependent on this one.

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1260733: Best Practices: constructors

Medium

Fix Released

Link a bug report

Related blueprints

260. By Felipe Figueiredo on 2013-12-16

* Changelog: File updated (fixes LP:1260733).

259. By Felipe Figueiredo on 2013-12-16

* lib/MMRRSim.pm (BUILDARGS, BUILD): Moved logger from BUILD() to
  BUILDARGS(). Removed finally empty BUILD() method. Now all attributes
  are defined in BUILDARGS into %attributes, and passed to the
  (now hidden and fully Moose'd) constructor.

258. By Felipe Figueiredo on 2013-12-16

* lib/MMRRSim.pm (BUILDARGS, BUILD): Moved basename definition from
  BUILD() to BUILDARGS().
  (_basename): New function, sets default value for basename, and calls
  _basename_postfix().

257. By Felipe Figueiredo on 2013-12-16

* lib/MMRRSim.pm (BUILDARGS, BUILD): Moved GridCell instantiation from
  BUILD() to BUILDARGS().

256. By Felipe Figueiredo on 2013-12-16

* lib/MMRRSim.pm (BUILDARGS, BUILD): Move required attributes definitions
  from BUILD() to BUILDARGS().

255. By Felipe Figueiredo on 2013-12-16

* lib/MMRRSim.pm (BUILDARGS, BUILD): New method BUILDARGS, creates config
  object before MMRRSim object creation. Code moved from BUILD(). Removed
  un-needed config_file attribute.
  (_basename_postfix): Changed from method to function.

254. By Felipe Figueiredo on 2013-12-16

* lib/MMRRSim/Config.pm: Re-add default for config file name.

253. By Felipe Figueiredo on 2013-12-16

* Makefile.PL: Add namespace::autoclean pre-requisite.

252. By Felipe Figueiredo on 2013-12-15

* lib/MMRRSim.pm (BUILD, init): Move logger object creation and
  configuration from init() to BUILD().

251. By Felipe Figueiredo on 2013-12-15

* lib/MMRRSim.pm: Minor organization of declarations. Move $VERSION to
  modules specific block.