-
Notifications
You must be signed in to change notification settings - Fork 241
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
Minor changes to Book weenie #490
Conversation
Would this not cause a conflict if I actually added an inscription to a book item? |
I wouldn't think so. They are the same data, see wcid 3949 |
What if I write in a Parchment (wcid 36), then give it to someone who adds an inscription? (Note that the handful of prewritten books I quickly checked were not inscribable, but blank ones are) |
In that case from a GameEventBookDataResponse perspective: inscription, authorId, authorName would reflect whomever inscribed the book, and each GameEventBookPageDataResponse would still contain whomever wrote each page |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall this is fine, but you've got a couple debug strings that need cleaning up.
@@ -29,7 +26,7 @@ public GameEventBookDataResponse(Session session, uint bookID, uint maxPages, Li | |||
if (session.Player.IsAdmin) | |||
Writer.WriteString16L(pageData[i].AuthorAccount); | |||
else | |||
Writer.WriteString16L(""); | |||
Writer.WriteString16L("beer good"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
???
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is what retail did when masking output, see pcaps
@@ -16,7 +15,7 @@ public GameEventBookPageDataResponse(Session session, uint bookID, PageData page | |||
if (session.Player.IsAdmin) | |||
Writer.WriteString16L(pageData.AuthorAccount); | |||
else | |||
Writer.WriteString16L(""); | |||
Writer.WriteString16L("Password is cheese"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
???
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is what retail did when masking output, see pcaps
No description provided.