Best Practices: attribute defaults

Bug #1260730 reported by Felipe Figueiredo
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
MMRRSim
Fix Released
Medium
Felipe Figueiredo

Bug Description

After fixing Bug #1256772, all that remains is to remove the cruft from the classes definitions of attributes. To this end, I must remove all "default" entry from the attributes.

Tags: cleanup

Related branches

Changed in mmrrsim:
milestone: 0.6.1.1 → 0.6.1.2
Changed in mmrrsim:
status: New → In Progress
Revision history for this message
Felipe Figueiredo (philsf) wrote :

In order to complete this bug fix, I need to address at least some portion of Bug LP:1260733. Defer final steps of attributes refactoring to after we are not modifying new() methods anymore.

Changed in mmrrsim:
status: In Progress → Triaged
Revision history for this message
Felipe Figueiredo (philsf) wrote :

Bug LP:1260733 is now closed, so a complete cleanup of object attributes is now possible.

Changed in mmrrsim:
status: Triaged → In Progress
Changed in mmrrsim:
status: In Progress → Triaged
Revision history for this message
Felipe Figueiredo (philsf) wrote :

Still missing:

- Move a_centers attribute to Config.pm
- Move logger initialization to Config.pm
- Call get_logger() from MMRRSim.

Revision history for this message
Felipe Figueiredo (philsf) wrote :

Still missing:

- Move logger initialization to Config.pm
- Call get_logger() from MMRRSim.

Revision history for this message
Felipe Figueiredo (philsf) wrote :

All changes committed to branch and merged into trunk.

Changed in mmrrsim:
status: Triaged → Fix Committed
Changed in mmrrsim:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.