MantisBT - Zandronum
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0002888Zandronum[All Projects] Bugpublic2016-10-12 22:042018-09-30 21:58
ReporterEnsaladaDeTomate 
Assigned ToDusk 
PriorityhighSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformMicrosoftOSWindowsOS VersionXP/Vista/7
Product Version2.0 
Target Version3.0Fixed in Version3.0 
Summary0002888: This recoil script has been broken since zandronum 2.0 came out
DescriptionThis bug wasnt happening before zandronum 2.0 was released, and since it seems that it has never been reported, it is still broken even with latest 3.0 build(s)

*Note - this bug currently occurs online, it works fine offline
Steps To Reproduce- create a custom server
- enable cheats
- join the game
- make sure 'switch on pickup' is enabled
- summon nailgun in console
- pick it
- shoot (epilepsy warning)
Additional InformationZandronum 1.0 - working fine both offline and online

Zandronum 1.1.1 - working fine both offline and online

Zandronum 1.2.1 - working fine both offline and online

Zandronum 2.0 + - only works offline, online is broken

Zandronum 3.0 + - same situation as 2.0
TagsNo tags attached.
Relationships
Attached Files? recoil.pk3 (32,994) 2016-10-12 22:04
/tracker/file_download.php?file_id=1937&type=bug
? recoil_2.pk3 (31,966) 2016-10-13 19:50
/tracker/file_download.php?file_id=1941&type=bug
? recoil_3.pk3 (31,626) 2016-10-13 20:54
/tracker/file_download.php?file_id=1942&type=bug
Issue History
Date ModifiedUsernameFieldChange
2016-10-12 22:04EnsaladaDeTomateNew Issue
2016-10-12 22:04EnsaladaDeTomateFile Added: recoil.pk3
2016-10-12 22:16DuskProduct Version3.0-beta => 2.0
2016-10-12 23:44Edward-sanSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=9702#r9702
2016-10-13 00:10unknownnaNote Added: 0015942
2016-10-13 00:10unknownnaSeverityminor => major
2016-10-13 00:10unknownnaStatusnew => confirmed
2016-10-13 00:28unknownnaPrioritynormal => high
2016-10-13 06:35Edward-sanNote Added: 0015948
2016-10-13 06:35Edward-sanStatusconfirmed => feedback
2016-10-13 07:29unknownnaNote Added: 0015952
2016-10-13 07:29unknownnaNote Edited: 0015952bug_revision_view_page.php?bugnote_id=15952#r9706
2016-10-13 19:45EnsaladaDeTomateNote Added: 0015961
2016-10-13 19:45EnsaladaDeTomateStatusfeedback => new
2016-10-13 19:50WaTaKiDFile Added: recoil_2.pk3
2016-10-13 20:27Edward-sanNote Added: 0015962
2016-10-13 20:27Edward-sanStatusnew => feedback
2016-10-13 20:53EnsaladaDeTomateNote Added: 0015964
2016-10-13 20:53EnsaladaDeTomateStatusfeedback => new
2016-10-13 20:54WaTaKiDFile Added: recoil_3.pk3
2016-10-13 21:04EnsaladaDeTomateNote Edited: 0015964bug_revision_view_page.php?bugnote_id=15964#r9712
2016-10-13 22:14DuskNote Added: 0015967
2016-10-13 22:14DuskAssigned To => Dusk
2016-10-13 22:14DuskStatusnew => needs review
2016-10-14 06:33Torr SamahoStatusneeds review => needs testing
2016-10-14 19:17WaTaKiDNote Added: 0015998
2016-10-14 19:55EnsaladaDeTomateNote Added: 0016001
2016-10-14 20:04Ru5tK1ngStatusneeds testing => resolved
2016-10-14 20:04Ru5tK1ngResolutionopen => fixed
2016-10-14 20:04Ru5tK1ngFixed in Version => 3.0
2016-10-14 20:04Ru5tK1ngTarget Version => 3.0
2018-09-30 21:58Blzut3Statusresolved => closed

Notes
(0015942)
unknownna   
2016-10-13 00:10   
I set the severity to major since it really messes up the view locally now. It works fine in 1.3 as well so it seems to be a 2.0 regression indeed.
(0015948)
Edward-san   
2016-10-13 06:35   
I think the acs code should be reduced more, it's hard to tell where the problem is happening. Otherwise bisecting is the only way for investigating.
(0015952)
unknownna   
2016-10-13 07:29   
The script seems to use SetActorPitch, if that helps.
(0015961)
EnsaladaDeTomate   
2016-10-13 19:45   
Hey, i tried to clean up and remove some unnecessary scripts from the file.

Since im not the main author of this script, it is pretty confusing for me to clean it even further, since im not familiar with it. I hope that this will help anyway.

'http://www.mediafire.com/file/gdc2uttdridta78/recoil_2.pk3 [^]'
(0015962)
Edward-san   
2016-10-13 20:27   
Still hard to understand where it's wrong. I'd like to see just one call of SetActorPitch in the whole script, if possible.
(0015964)
EnsaladaDeTomate   
2016-10-13 20:53   
(edited on: 2016-10-13 21:04)
try this'http://www.mediafire.com/file/h9z7m14nddwhn2g/recoil_3.pk3 [^]'

Edit: Just tested with zandronum 1.3, and the difference is pretty clear, so this example wad should be good.
(0015967)
Dusk   
2016-10-13 22:14   
Fixed: this was a problem in underlying networking code that flew past the radar.
(0015998)
WaTaKiD   
2016-10-14 19:17   
this 3.0 build contains the above commit: 'https://www.dropbox.com/s/0mgvkwxy75hymom/zandronum-3.0-r161013-2149-dda63e8-windows.zip?dl=0 [^]'
(0016001)
EnsaladaDeTomate   
2016-10-14 19:55   
Just tested it, and the bug seems to be gone, so this can be marked as resolved.