MantisBT - Zandronum
View Issue Details
0000721Zandronum[All Projects] Bugpublic2012-03-25 23:072018-09-30 19:53
Dusk 
Dusk 
normalminoralways
closedfixed 
1.0-beta 
1.0 
0000721: Mystic Ambit Incant broken online
The Mystic Ambit Incant seems to be causing desyncs.

I'll look into this soon enough.
Hexen
Issue History
2012-03-25 23:07DuskNew Issue
2012-03-25 23:07DuskStatusnew => assigned
2012-03-25 23:07DuskAssigned To => Dusk
2012-03-29 15:52DuskSummaryMystic Ambit Incant broken online with the mage => Mystic Ambit Incant broken online
2012-03-29 15:52DuskDescription Updatedbug_revision_view_page.php?rev_id=1553#r1553
2012-03-29 15:53DuskNote Added: 0003005
2012-03-29 15:53DuskStatusassigned => feedback
2012-03-29 15:54DuskReproducibilityhave not tried => always
2012-03-31 16:49Torr SamahoNote Added: 0003064
2012-03-31 17:10DuskNote Added: 0003066
2012-03-31 17:10DuskStatusfeedback => assigned
2012-03-31 17:10DuskStatusassigned => feedback
2012-03-31 17:51Torr SamahoNote Added: 0003068
2012-03-31 20:16DuskNote Added: 0003072
2012-03-31 20:16DuskStatusfeedback => assigned
2012-04-01 14:26Torr SamahoNote Added: 0003076
2012-04-01 14:29DuskNote Added: 0003077
2012-04-01 14:30DuskStatusassigned => feedback
2012-04-16 02:37DuskNote Added: 0003317
2012-04-16 02:37DuskStatusfeedback => assigned
2012-04-16 02:37DuskStatusassigned => resolved
2012-04-16 02:37DuskFixed in Version => 1.0-beta
2012-04-16 02:37DuskResolutionopen => fixed
2012-04-20 22:41Blzut3Fixed in Version1.0-beta => 1.0
2012-06-09 13:22Torr SamahoCategoryGeneral => Bug
2018-09-30 19:53Blzut3Statusresolved => closed

Notes
(0003005)
Dusk   
2012-03-29 15:53   
'https://bitbucket.org/CrimsonDusk/notebola/changeset/e3be483bcb99 [^]'

What a frustrating one, I kept doing it doing it wrong! :(
It should work now.
(0003064)
Torr Samaho   
2012-03-31 16:49   
Small remarks:

- The "( NETWORK_GetState() == NETSTATE_CLIENT )" check should be replaced by NETWORK_InClientMode() (otherwise your fix doesn't work when playing a demo)
- The second argument in SERVERCOMMANDS_SoundActor should be CHAN_AUTO instead of ATTN_NORM.

Would be nice if you can make these changes and provide testing binaries so that somebody can confirm that this is working (except for the two small things mentioned above the patch looks very good to me, but it would be nice if it gets a bit more testing).
(0003066)
Dusk   
2012-03-31 17:10   
Fixed derps:'https://bitbucket.org/CrimsonDusk/notebola/changeset/24f83550d6b7 [^]'

As for testing binaries, I could try make 32-bit Linux ones but Windows ones I have no clue how to make.
(0003068)
Torr Samaho   
2012-03-31 17:51   
> As for testing binaries, I could try make 32-bit Linux ones but Windows ones I have no clue how to make.

I see, I don't want to overcomplicate things. So I'll just assume you tested your patch thoroughly and add it, ok?

Before closing the issue, some more testing should be done then with the next beta build.
(0003072)
Dusk   
2012-03-31 20:16   
Hm, I'll check demo playback first to ensure nothing should be wrong.
(0003076)
Torr Samaho   
2012-04-01 14:26   
Patch added. So the only thing still missing is thorough testing with the next beta build.
(0003077)
Dusk   
2012-04-01 14:29   
OK, nice. :)

I'll mark this as needing feedback. When the beta is out, I'll try organize some Hexen testing and see how it works.
(0003317)
Dusk   
2012-04-16 02:37   
Fix confirmed to work in the latest beta.