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

Dat Path Updates #164

Merged
merged 2 commits into from
Mar 22, 2017
Merged

Conversation

fantoms
Copy link
Contributor

@fantoms fantoms commented Mar 22, 2017

Update to Yesterday's Path support PR. The path support push broke the loading of dats without trailing slashes in the config. Here are the changes in this PR that address this issue:

  • Updated example DatFileDirectory with a trailing slash at the end of the value

  • Changed DatLoader to use Path.Combine, trailing slash is no longer needed.

ServerTime initialized to Date: Wintersebb 22, 28 P.Y. Time: Warmtide 2017-03-22 10:04:22,672 INFO : Successfully opened C:\ACE\client_cell_1.dat file, containing 1 records 2017-03-22 10:04:22,781 INFO : Successfully opened C:\ACE\client_portal.dat file, containing 1 records

@Mogwai-TheFurry Mogwai-TheFurry merged commit c066c1b into ACEmulator:master Mar 22, 2017
ogmage78 pushed a commit to ogmage78/ACE that referenced this pull request Mar 24, 2017
* Updated example DatFileDirectory with a trailing slash at the end of the value

* Added Path.Combine to DatLoader

Finished fixing merge

Update for merge

- WIP Drop Item - getting the motion packet - slow but sure

- WIP Drop Item

WIP on drop item - adding mising enum

Still not doing this right

Drop Item - Getting oh so close to making this work

Still working on Drop Item

pcap looks the same - he is not bending over and the wand does not appear.   He did move once or twice, not sure what I changed.
@fantoms fantoms deleted the fix-trailingDatSlash branch May 23, 2017 15:50
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

2 participants