DevZenDao - Voting bug

[AnthonyAkentiev]

вообщем так и не поправил этот баг
when "add new group member" voting is created then on the last vote, that would add team member, metamask consumes huge amount of gas so there is smth wrong

после создания голосования на добавление нового участника через daoBaseAuto.addGroupMemberAuto(), если к примеру есть 3 участника в группе и первый голосует "за", то при голосовании второго выскакивает revert

truffle debug ссылается на:

263:   * @dev this function issue tokens for address _to
264:   */
265:   function issueTokens(address _tokenAddress, address _to, uint _amount) public isCanDo(ISSUE_TOKENS) {
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

debug(development:0x79ba65ba...)> 

Transaction halted with a RUNTIME ERROR.

This is likely due to an intentional halting expression, like assert(), require() or revert(). It can also be due to out-of-gas exceptions. Please inspect your transaction parameters and contract code to determine the meaning of this error.```
насколько я понимаю isCanDo возвращает false и поэтому выскакивает revert, но причем здесь permission на ISSUE_TOKENS я так и не понял, в теории чтобы добавить нового участника в группу этот permission вообще не нужен

вполне возможно, что в core то же самое, так как нет функциональных тестов, которые проверяют "полный круг" голосования: создание => голосование => проверка результатов, такие тесты по идее должны быть тут https://github.com/Thetta/Thetta-DAO-Framework/blob/master/test/daobase_auto.functional.tests.js

был такой таск https://github.com/Thetta/Thetta-DAO-Framework/issues/277, но он теперь закрыт, вероятно следует его открыть заново

из полезного поправил небольшие косяки на фронте DevZenDao и добавил читабельные параметры для каждого предложения

[ryzhak]

this test reverts on voting:

describe("addGroupMemberAuto", () => {
		it("should add a new group member", async() => {
			await devZenDaoAuto.addGroupMemberAuto("DevZenTeam", guest1, {from: boss}).should.be.fulfilled;

			const voting = await getVoting(daoBase, 0);
			await voting.vote(true, {from: teamMember1});

			console.log(await voting.getVotingStats());
			console.log(await voting.isFinished());
			console.log(await voting.isYes());
		});
	});