Kate Ross OrgChart

Do I have to return it in with those new four lines?


On Fri, Dec 12, 2014 at 8:07 AM, Don Irwin <notifications@github.com> wrote:

> In src/Employee.java
> <https://github.com/BASISPhoenix/12-10-OrgChart/pull/12#discussion-diff-21749000>
> :
>
> > +  }
> > +
> > +  public void setHours(int h) {
> > +    hoursWorked = h;
> > +  }
> > +
> > +  public void setJobTitle(String title) {
> > +    jobTitle = title;
> > +  }
> > +
> > +  public void giveRaise() {
> > +    hourlyWage ++;
> > +  }
> > +
> > +  public void setManager(Manager newManager) {
> > +    manager = newManager;
>
> incomplete - need to add to manager's list:
>
>          if(this.manager != null) {
>              this.manager.removeDirectReport(this);
>          }
>          this.manager = manager;
>          manager.addDirectReport(this);
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BASISPhoenix/12-10-OrgChart/pull/12/files#r21749000>.
>



-- 
Kate Ross
(\__/)
(='.'=)