add upgrade plan event

[pause125]

Pull request type

Please check the type of change your PR introduces:

• Bugfix
• Feature
• Code style update (formatting, renaming)
• Refactoring (no functional changes, no api changes)
• Build related changes
• Documentation content changes
• Other (please describe):

What is the current behavior?

Issue Number: resolve #11

What is the new behavior?

Other information

[pause125]

这里后向兼容的考虑应该是放到 submit_upgrade_plan_with_cap_v2 里面，但是这样需要 submit_upgrade_plan_with_cap_v2 接受 signer 参数。
这种兼容问题一般怎么处理？@jolestar

[jolestar]

这里要考虑到旧的已经升级成 TwoPhase 的账户没有 UpgradePlanEventHolder, 需要做个兼容处理。但这里拿不到 package address 的 signer，能想到两个办法：

1. 提供一个初始化 UpgradePlanEventHolder 的入口方法，旧的地址调用一下
2. 通过 Account module 提供一个 friend 方法来模拟用户创建和写入 UpgradePlanEventHolder

这个兼容性问题怎么测试也需要想一下.

[WGB5445]

这里要考虑到旧的已经升级成 TwoPhase 的账户没有 UpgradePlanEventHolder, 需要做个兼容处理。但这里拿不到 package address 的 signer，能想到两个办法：

1. 提供一个初始化 UpgradePlanEventHolder 的入口方法，旧的地址调用一下
2. 通过 Account module 提供一个 friend 方法来模拟用户创建和写入 UpgradePlanEventHolder

这个兼容性问题怎么测试也需要想一下.

我感觉这里还是第二种方式比较好：
1.submit_upgrade_plan_v2(account: &signer, package_hash: vector<u8>, version:u64, enforced: bool)一般由个人/多签账户发起，此时可以初始化UpgradePlanEventHolder
2.submit_upgrade_plan_with_cap_v2(cap: &UpgradePlanCapability, package_hash: vector<u8>, version: u64, enforced: bool)一般由 signer托管账户(DAO账户)调用，如果只提供单独的 初始化入口方法， DAO 账户的合约将无法升级

[pause125]

方案二 可以给任意地址新增 UpgradePlanEventHolder 资源吗？

[pause125]

测试的话是否可以从 banard fork 状态，0x1 地址上应该已经有 TwoPhaseUpgradeV2 但是没有 UpgradePlanEventHolder。符合测试条件。

[WGB5445]

方案二 可以给任意地址新增 UpgradePlanEventHolder 资源吗？

用地址生成一个 signer ，“替用户执行”

[jolestar]

方案二 可以给任意地址新增 UpgradePlanEventHolder 资源吗？

friend, 仅 framework 内部调用

[WGB5445]

方案二 可以给任意地址新增 UpgradePlanEventHolder 资源吗？

用地址生成一个 signer ，“替用户执行”

参考这个函数的使用方法:

    public (friend) fun get_genesis_capability():SignerCapability{
        let signer_cap = SignerCapability {addr: Token::token_address<STC>() };
        signer_cap
    }

https://github.com/starcoinorg/starcoin-framework/blob/main/sources/Account.move#L19

[pause125]

方案二会产生循环引用，PackageTxnManager use Account, Account use STC, STC use PackageTxnManager .

[jolestar]

方案二会产生循环引用，PackageTxnManager use Account, Account use STC, STC use PackageTxnManager .

哦，这是个问题。初始化旧版本的 DAO 的逻辑，感觉需要清理一下，那感觉可以先用方案一，现在通过 signer 托管方式实现的应该少，原来的 signer 托管后有升级合约的难题。

[jolestar]

清理的问题记录了一个 issue #139

[yubing744]

这个IF判断感觉可以去掉？

[pause125]

fixed