[BUG] injection of self and world into dojo contract should be scoped

[glihm]

Describe the bug
Currently, when an #[dojo::interface] is defined, we inject automatically the self: @ContractState into the functions it contains. Same for the world, if a function contains world: IWorldDispatcher as it's first argument, we inline it.

The issue with that, is when we have internal functions or standalone function, we're applying this rule everywhere.

To Reproduce
Write a simple function into a contract that takes no argument, and when you try to call it, it will ask for an argument (being self). Also, if you have an internal implementation with a function that takes world: IWorldDispatcher as argument, it will be removed and cause compilation error.

Expected behavior
The inlining of the world: IWorldDispatcher and the auto management of world: IWorldDispatcher must only be done for #[dojo::interfaces] implementations.

The related code for contract macro is here:

dojo/crates/dojo-lang/src/contract.rs

Line 223 in 48e7261

pub fn check_self_parameter(

And the one for interfaces here:

dojo/crates/dojo-lang/src/interface.rs

Line 81 in 48e7261

pub fn rewrite_parameters(

[glihm]

Closed in #1733.