Skip to content

Remove invalid company session setter#233

Merged
choran merged 1 commit intomasterfrom
timlim/fix-company-sessions
Oct 9, 2018
Merged

Remove invalid company session setter#233
choran merged 1 commit intomasterfrom
timlim/fix-company-sessions

Conversation

@thewheat
Copy link
Contributor

@thewheat thewheat commented Oct 3, 2018

Why?

How?

  • Remove references to setSessionCount for companies

Test Code that will fail on current code base but will succeed with these new changes

        HashMap<String, String> map = Maps.newHashMap();
        map.put("company_id", "1");
        Company company = Company.find(map);
        company.setName("Blue Sun Corporation");
        Company.update(company);

        map = Maps.newHashMap();
        map.put("user_id", "1");
        User user = User.find(map);
        user.addCompany(company);
        User.update(user);

@choran choran added the java label Oct 9, 2018
@choran choran merged commit 6678c17 into master Oct 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants