added comments that still need to be edited

This commit is contained in:
Griff Green 2017-07-09 12:04:52 -05:00 committed by GitHub
parent 3016960a47
commit 4410f0b045
1 changed files with 60 additions and 41 deletions

View File

@ -5,22 +5,24 @@ import "./LiquidPledgingBase.sol";
contract LiquidPledging is LiquidPledgingBase { contract LiquidPledging is LiquidPledgingBase {
uint constant MAX_DELEGATES = 20;
uint constant MAX_SUBPROJECT_LEVEL = 20;
////// //////
// Constructor // Constructor
////// //////
// This constructor actualy also calls the constructor for the
// `LiquidPledgingBase` contract
function LiquidPledging(address _vault) LiquidPledgingBase(_vault) { function LiquidPledging(address _vault) LiquidPledgingBase(_vault) {
} }
/// @notice This is the main entry of Ether to the system. Ethether goes to /// @notice This is how value enters into the system which creates notes. The
/// the vault and then the Note for the donor withou delegates is increased. /// token of value goes into the vault and then the amount in the Note
/// relevant to this donor without delegates is increased.
/// After that, a normal transfer is done to the idReceiver. /// After that, a normal transfer is done to the idReceiver.
/// @param idDonor Id of the donor thats donating. /// @param idDonor Identifier of the donor thats donating.
/// @param idReceiver To who it's transfered. Can ve the same donnor, another /// @param idReceiver To whom it's transfered. Can be the same donor, another
/// donor, a delegate or a project /// donor, a delegate or a project
function donate(uint64 idDonor, uint64 idReceiver) payable { function donate(uint64 idDonor, uint64 idReceiver) payable {// TODO change to `pledge()`
NoteManager sender = findManager(idDonor); NoteManager sender = findManager(idDonor);
if (sender.managerType != NoteManagerType.Donor) throw; if (sender.managerType != NoteManagerType.Donor) throw;
@ -30,10 +32,10 @@ contract LiquidPledging is LiquidPledgingBase {
if (amount == 0) throw; if (amount == 0) throw;
vault.transfer(amount); vault.transfer(amount); // transfers the baseToken to the Vault
uint64 idNote = findNote( uint64 idNote = findNote(
idDonor, idDonor,
new uint64[](0), new uint64[](0), //what is new
0, 0,
0, 0,
0, 0,
@ -49,13 +51,13 @@ contract LiquidPledging is LiquidPledgingBase {
} }
/// @notice This is the main function to move Ether from one Note to the other /// @notice This is the main function to move value from one Note to the other
/// @param idSender Id of the donor, delegate or project manager that is transferin /// @param idSender ID of the donor, delegate or project manager that is transfering
/// the funds from Note to note. This manager must have permisions to move the Ether /// the funds from Note to Note. This manager must have permisions to move the value
/// @param idNote Id of the note that's moving the Ether. /// @param idNote Id of the note that's moving the value
/// @param amount Quantity of Ether that's moving. /// @param amount Quantity of value that's being moved
/// @param idReceiver Destination of the Ether, can be a donor to move Ether between donors, /// @param idReceiver Destination of the value, can be a donor sending to a donor or
/// A delegate to delegate that Ether, or a project to commit or precommit it to that project. /// a delegate, a delegate to another delegate or a project to precommit it to that project
function transfer(uint64 idSender, uint64 idNote, uint amount, uint64 idReceiver) { function transfer(uint64 idSender, uint64 idNote, uint amount, uint64 idReceiver) {
idNote = normalizeNote(idNote); idNote = normalizeNote(idNote);
@ -85,7 +87,7 @@ contract LiquidPledging is LiquidPledgingBase {
uint senderDIdx = getDelegateIdx(n, idSender); uint senderDIdx = getDelegateIdx(n, idSender);
if (senderDIdx != NOTFOUND) { if (senderDIdx != NOTFOUND) {
// If the receiver is another doner // If the receiver is another donor
if (receiver.managerType == NoteManagerType.Donor) { if (receiver.managerType == NoteManagerType.Donor) {
// Only accept to change to the original donor to remove all delegates // Only accept to change to the original donor to remove all delegates
if (n.owner == idReceiver) { if (n.owner == idReceiver) {
@ -99,23 +101,32 @@ contract LiquidPledging is LiquidPledgingBase {
// If the receiver is another delegate // If the receiver is another delegate
if (receiver.managerType == NoteManagerType.Delegate) { if (receiver.managerType == NoteManagerType.Delegate) {
uint receiverDIdx = getDelegateIdx(n, idReceiver); uint receiverDIdx = getDelegateIdx(n, idReceiver);
// If the receiver is not in the delegate list // If the receiver is not in the delegate list
if (receiverDIdx == NOTFOUND) { if (receiverDIdx == NOTFOUND) {
undelegate(idNote, amount, n.delegationChain.length - senderDIdx - 1); undelegate(idNote, amount, n.delegationChain.length - senderDIdx - 1);
appendDelegate(idNote, amount, idReceiver); appendDelegate(idNote, amount, idReceiver);
// If the receiver is after the delegate list and is not the next one.
// Canccel delegations an redelegate // If the receiver is already part of the delegate chain and is
// after the sender, then all of the other delegates after the sender are
// removed and the receiver is appended at the end of the delegation chain
} else if (receiverDIdx > senderDIdx) { } else if (receiverDIdx > senderDIdx) {
undelegate(idNote, amount, n.delegationChain.length - senderDIdx - 1); undelegate(idNote, amount, n.delegationChain.length - senderDIdx - 1);
appendDelegate(idNote, amount, idReceiver); appendDelegate(idNote, amount, idReceiver);
// If it's before the list cancel thelegations until him
// If the receiver is already part of the delegate chain and is
// before the sender, then the sender and all of the other
// delegates after the RECEIVER are revomved from the chain,
// this is interesting because the delegate undelegates from the
// delegates that delegated to this delegate... game theory issues? should this be allowed
} else if (receiverDIdx <= senderDIdx) { } else if (receiverDIdx <= senderDIdx) {
undelegate(idNote, amount, n.delegationChain.length - receiverDIdx -1); undelegate(idNote, amount, n.delegationChain.length - receiverDIdx -1);
} }
return; return;
} }
// If the delegate chose a project to assign // If the delegate wants to support a project, they undelegate all
// the delegates after them in the chain and choose a project
if (receiver.managerType == NoteManagerType.Project) { if (receiver.managerType == NoteManagerType.Project) {
undelegate(idNote, amount, n.delegationChain.length - senderDIdx - 1); undelegate(idNote, amount, n.delegationChain.length - senderDIdx - 1);
proposeAssignProject(idNote, amount, idReceiver); proposeAssignProject(idNote, amount, idReceiver);
@ -126,8 +137,8 @@ contract LiquidPledging is LiquidPledgingBase {
} }
/// @notice This method is used to withdraw Ether from the system. This can be used /// @notice This method is used to withdraw value from the system. This can be used
/// from the doonurs to rollback a not commited donation or by project manager to use /// by the donors to avoid committing the donation or by project manager to use
/// the Ether. /// the Ether.
/// @param idNote Id of the note that wants to be withdrawed. /// @param idNote Id of the note that wants to be withdrawed.
/// @param amount Quantity of Ether that wants to be withdrawed. /// @param amount Quantity of Ether that wants to be withdrawed.
@ -186,7 +197,7 @@ contract LiquidPledging is LiquidPledgingBase {
function cancelPayment(uint64 idNote, uint amount) onlyVault { function cancelPayment(uint64 idNote, uint amount) onlyVault {
Note n = findNote(idNote); Note n = findNote(idNote);
if (n.paymentState != PaymentState.Paying) throw; if (n.paymentState != PaymentState.Paying) throw; //TODO change to revert
// When a payment is cacnceled, never is assigned to a project. // When a payment is cacnceled, never is assigned to a project.
uint64 oldNote = findNote( uint64 oldNote = findNote(
@ -215,6 +226,8 @@ contract LiquidPledging is LiquidPledgingBase {
// Multi note methods // Multi note methods
//////// ////////
// This set of functions makes moving a lot of notes around much more
// efficient (saves gas) than calling these functions in series
uint constant D64 = 0x10000000000000000; uint constant D64 = 0x10000000000000000;
function mTransfer(uint64 idSender, uint[] notesAmounts, uint64 idReceiver) { function mTransfer(uint64 idSender, uint[] notesAmounts, uint64 idReceiver) {
for (uint i = 0; i < notesAmounts.length; i++ ) { for (uint i = 0; i < notesAmounts.length; i++ ) {
@ -256,7 +269,8 @@ contract LiquidPledging is LiquidPledgingBase {
// Private methods // Private methods
/////// ///////
// this function is obvious, but it can also be called to undelegate everyone
// by setting your self as teh idReceiver
function transferOwnershipToProject(uint64 idNote, uint amount, uint64 idReceiver) internal { function transferOwnershipToProject(uint64 idNote, uint amount, uint64 idReceiver) internal {
Note n = findNote(idNote); Note n = findNote(idNote);
@ -268,24 +282,18 @@ contract LiquidPledging is LiquidPledgingBase {
0, 0,
n.oldNote, n.oldNote,
PaymentState.NotPaid); PaymentState.NotPaid);
// If the owner does not change, then just let it this way.
if (n.owner == idReceiver) return;
uint64 toNote = findNote( uint64 toNote = findNote(
idReceiver, idReceiver,
new uint64[](0), new uint64[](0),
0, 0,
0, 0,
oldNote, oldNote,
PaymentState.NotPaid); PaymentState.NotPaid);
doTransfer(idNote, toNote, amount); doTransfer(idNote, toNote, amount);
} }
function transferOwnershipToDonor(uint64 idNote, uint amount, uint64 idReceiver) internal { function transferOwnershipToDonor(uint64 idNote, uint amount, uint64 idReceiver) internal {
// If the owner does not change, then just let it this way.
Note n = findNote(idNote); Note n = findNote(idNote);
if (n.owner == idReceiver) return;
uint64 toNote = findNote( uint64 toNote = findNote(
idReceiver, idReceiver,
new uint64[](0), new uint64[](0),
@ -299,12 +307,15 @@ contract LiquidPledging is LiquidPledgingBase {
function appendDelegate(uint64 idNote, uint amount, uint64 idReceiver) internal { function appendDelegate(uint64 idNote, uint amount, uint64 idReceiver) internal {
Note n = findNote(idNote); Note n = findNote(idNote);
if (n.delegationChain.length >= MAX_DELEGATES) throw; if (n.delegationChain.length >= MAX_DELEGATES) throw; //TODO change to revert and say the error
uint64[] memory newDelegationChain = new uint64[](n.delegationChain.length + 1); uint64[] memory newDelegationChain = new uint64[](n.delegationChain.length + 1);
for (uint i=0; i<n.delegationChain.length; i++) { for (uint i=0; i<n.delegationChain.length; i++) {
newDelegationChain[i] = n.delegationChain[i]; newDelegationChain[i] = n.delegationChain[i];
} }
// Make the last item in the array the idReceiver
newDelegationChain[n.delegationChain.length] = idReceiver; newDelegationChain[n.delegationChain.length] = idReceiver;
uint64 toNote = findNote( uint64 toNote = findNote(
n.owner, n.owner,
newDelegationChain, newDelegationChain,
@ -333,7 +344,7 @@ contract LiquidPledging is LiquidPledgingBase {
} }
function proposeAssignProject(uint64 idNote, uint amount, uint64 idReceiver) internal { function proposeAssignProject(uint64 idNote, uint amount, uint64 idReceiver) internal {// Todo rename
Note n = findNote(idNote); Note n = findNote(idNote);
if (getProjectLevel(n) >= MAX_SUBPROJECT_LEVEL) throw; if (getProjectLevel(n) >= MAX_SUBPROJECT_LEVEL) throw;
@ -361,8 +372,16 @@ contract LiquidPledging is LiquidPledgingBase {
Transfer(from, to, amount); Transfer(from, to, amount);
} }
// This function does 2 things, #1: it checks to make sure that the pledges are correct
// if the a pledged project has already been commited then it changes the owner
// to be the proposed project (Note that the UI will have to read the commit time and manually
// do what this function does to the note for the end user at the expiration of the committime)
// #2: It checks to make sure that if there has been a cancellation in the chain of projects,
// then it adjusts the note's owner appropriately.
function normalizeNote(uint64 idNote) internal returns(uint64) { function normalizeNote(uint64 idNote) internal returns(uint64) {
Note n = findNote(idNote); Note n = findNote(idNote);
// Check to make sure this note hasnt already been used or is in the process of being used
if (n.paymentState != PaymentState.NotPaid) return idNote; if (n.paymentState != PaymentState.NotPaid) return idNote;
// First send to a project if it's proposed and commited // First send to a project if it's proposed and commited
@ -386,7 +405,7 @@ contract LiquidPledging is LiquidPledgingBase {
n = findNote(idNote); n = findNote(idNote);
} }
toNote = getOldestNoteNotCanceled(idNote); toNote = getOldestNoteNotCanceled(idNote);// TODO toNote is note defined
if (toNote != idNote) { if (toNote != idNote) {
doTransfer(idNote, toNote, n.amount); doTransfer(idNote, toNote, n.amount);
} }