Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[NOT COMPLETE] Draft of UO Steam support implementation #1

Open
wants to merge 102 commits into
base: master
Choose a base branch
from

Conversation

diogo-strube
Copy link
Owner

@diogo-strube diogo-strube commented Feb 9, 2021

The current Draft has a few interesting points:

  • Use of generics and dependency injection to increase code reuse;
  • Alias beyond Serial allows defining mapping per argument/parameter.
  • ParametertList encapsulates the handling of optional fields and array arguments.
  • Queues for execution are automatically created

There are in reality a few classes and three commits, but me moving between forks/branched made this PR look terrible. Commits are:

@diogo-strube draft of implementation... 7fa5f70
@diogo-strube drafting UO Steam impleme... 0b00779
@diogo-strube added context alias, comma... b929a35

Overall, support for all commands should already be possible with implemented engine. And it would be epic if we could go beyond that.

Storm Kiernan and others added 30 commits January 4, 2021 20:51
These will be useful later.
2. adding a new find item function to the player that includes hue, range, graphic
3. finishing the "usetype" command
2. adding support for "useobject"
3. adding support for "moveitem"
…; the timing already took this into consideration.
… position when clicked and its real position in the space. When moving mouse quickly while dragging a gump the function got the current mouse position and not the clicked one. In low end PCs this will cause a mismatch.
diogo-strube and others added 26 commits February 16, 2021 19:30
…er now does not call Commands internal logic
…ht away and not queued and adding support for default optional strings
…ieving file from revision 3c1a8ffa519f847d827b78aa2e91252df2b8224
…logic in ArgumentList to allow for non-serial value mapping.
…be converted as a Serial and added support to the Alias commands
…nds logic:BandageSelf, FindType, UseType, UseObject, UseOnce, Turn, Run
…. Also fixed a few issues while testing commands, such as wait of expression and convertion of virtual arguments
…ging the GameActions.cs file logic to comply with UO Steam syntax, going beyond script related source code
…e return false and timeout; 2) Changing time logic to use Tick; 3) Fixing small bus across the code; 4) Adding md file with architecture and design notes.
…uipping items; 3) Fixed argument mapping bug; 4) Added msg related commands
… and a few list commands.

Code will be further updated to match new strucutre wit the ArgumentList.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants