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

Creature kill corpse v2 #239

Merged
merged 7 commits into from
Apr 18, 2017

Conversation

Lidefeath
Copy link
Contributor

This is the new version of the creature kill and corpse spawning with the smite command. Now the animations work without a sleep command but by introducing delays into the GameActionQueue. More details in the changelog.

fantoms added a commit to fantoms/ACE that referenced this pull request Apr 17, 2017
Copy link
Contributor

@Mogwai-TheFurry Mogwai-TheFurry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So, I didn't finish the review, and here's why:

You've intermixed a necessary feature (dropping of a corpse) with an unnecessary admin command (smite) where a "/testcorpsedrop" would have both been better and have gotten the job done.

// TODO: output
if (parameters?.Length > 0)
{
if (parameters[0] == "all")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The more I think about it, the more I don't like having "/smite all" as a thing. Not needed by players, not needed by admins (really), and creates some unnecessary design complications.

@Lidefeath
Copy link
Contributor Author

I removed the smite command as requested - although I don't see why implementing it was so bad if it's in the client.

@Mogwai-TheFurry Mogwai-TheFurry merged commit c083b22 into ACEmulator:master Apr 18, 2017
Lidefeath added a commit to Lidefeath/ACE that referenced this pull request Apr 18, 2017
Despawn corpses and respawn creatures work. Improved the spawn command in debugcommands.cs.

Spawn creatures in landblock from reading the generator data in db
@Lidefeath Lidefeath deleted the creature-kill-corpse branch May 2, 2017 12:13
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.

None yet

3 participants