Bugfix transaction multiple function declaration

[devbugging]

Closes #437

Description

This PR fixes a bug where a transaction includes multiple function declarations due to nature of SoleTransactionDeclaration.

Example of such transaction is:

pub fun dummy(_ address: Address): Void {}

transaction {
  prepare(acct: AuthAccount) {}
}

---

For contributor use:

• Targeted PR against master branch
• Linked to Github issue with discussion and accepted design OR link to spec that describes this work
• Code follows the standards mentioned here
• Updated relevant documentation
• Re-reviewed Files changed in the Github PR explorer
• Added appropriate labels

[janezpodhostnik]

The changes look good.

I'm wondering if it would make sense to change how SoleTransactionDeclaration works instead, so that his would be possible. I guess this is question for @Bastian @turbolent and definitely not in scope here.

edit: Sorry for the ping other Bastian!

[Bastian]

I guess this is question for @Bastian

I disagree 😉

[devbugging]

The changes look good.

I'm wondering if it would make sense to change how SoleTransactionDeclaration works instead, so that his would be possible. I guess this is question for @Bastian and definitely not in scope here.

@janezpodhostnik you probably mean @turbolent
I assumed from the naming of the func that this is indeed how it should work but yeah maybe Bastian can share some light.