Skip to content

prevent tokens from being sent to the dao itself #197

Merged
merged 2 commits into from Jun 3, 2016

4 participants

@p-s-dev
p-s-dev commented May 30, 2016

No description provided.

@CJentzsch

why?

@D-Nice
D-Nice commented May 30, 2016

It dilutes the overall value of tokens, and they won't contribute to votes.

@p-s-dev
p-s-dev commented May 30, 2016
  1. People send on accident, thinking that is how to vote
  2. mist doesn't sum tokens correctly when dao holds them (granted, that is a mist "bug")
@p-s-dev
p-s-dev commented Jun 1, 2016

Currently, people have accidentally sent 56,828 dao tokens (worth ~$7,330) to TheDao.

@CJentzsch CJentzsch and 1 other commented on an outdated diff Jun 2, 2016
@@ -740,6 +740,7 @@ contract DAO is DAOInterface, Token, TokenCreation {
if (isFueled
&& now > closingTime
&& !isBlocked(msg.sender)
+ && _to != address(this)
@CJentzsch
CJentzsch added a note Jun 2, 2016

you would allso need that change in transferFrom

@p-s-dev
p-s-dev added a note Jun 3, 2016

Done, added to transferFrom() also

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@CJentzsch CJentzsch merged commit 6306b34 into slockit:master Jun 3, 2016
@LefterisJP LefterisJP added this to the DAO v1.1 milestone Jun 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.