Fixes Camera.project_position to not return get_global_transform().origin if projection mode is orthogonal

[fsy98]

Fix #33837

[akien-mga]

Could you squash commits together, and give the resulting commit a more explicit commit message? See https://github.com/godotengine/godot/blob/master/CONTRIBUTING.md#format-your-commit-logs-with-readability-in-mind and PR workflow.

[fsy98]

is wired that I squash commits together,but it make mistake

[akien-mga]

You still have 2 commits, with Merge pull request #1 from godotengine/master which should not be included in this PR.

And the commit message for the second commit is:

Update camera.cpp

Update camera.cpp

which is not explicit at all.

[fsy98]

Sorry I am totally new to the GitHub, I force change the commit message, but I am confused why after I did the squash commits, it still shows Merge pull request #1 from godotengine/master …

[fsy98]

I don't know why I am in that commit, is not done by me, and it seams that is the master of my commit, did any one have a good idea to solve that?

[akien-mga]

Sorry for the delay.

The first commit is there because one of your team member used a Pull Request to update your fork (Team-Bad#1) instead of doing a manual pull. GitHub sadly lacks an option in its interface to update a branch without custom modifications to the latest upstream master branch.

I fixed it myself by rebasing on the upstream master branch, and fixed the style issues that were remaining (missing space before !=, and trailing tabs after }). I could not force-update the PR as modifications from maintainers have been disabled, so I pushed the fixed commit directly: d80e979.

Thanks for the contribution!