[STCC-229] variables not referenced correctly

[oiduh]

Previously, if a global variable was referenced in a sprite (that is not the background), a new local variable with the same name was added to the sprite. As a result the pocket code program could only access the local variant of the variable and scripts didn't work as they were intended to.
Since the background in scratch has no local variables, it serves as a container for all global variables. In the code all global variables are stored in self.project.userVariables and local variables in self.sprite.userVariables. In case a variable within a sprite cant be accessed, it has to be a global one and it has to be referenced correctly, so the script can use it.
In order to prevent confusion in the future, comments were added to explain how to access the variables.

[gregbauer]

This does not work as there is no possibility for sprites other than the background sprite to set/change global variables. The check "catrobat.is_background_sprite(self.sprite)" may be redundant here. Just ask for local variables first (as these probably shadow global variables? Needs to be tested?) and then check for global variables (in all sprites, not only the background sprite)
Here is the project I tested your changes with: https://scratch.mit.edu/projects/380191404/

Also, I think you should remove the config.ini from the commit.?

[oiduh]

You're right. I changed the code accordingly and adjusted the comments. The project you used initially for the test should also work now correctly.