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.

Sign up for GitHub

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

Jump to bottom

RscAttributeHeadlight.sqf suspension error with non-local vehicles #202

Closed
McDiod opened this issue Dec 16, 2017 · 0 comments
Closed

RscAttributeHeadlight.sqf suspension error with non-local vehicles #202

McDiod opened this issue Dec 16, 2017 · 0 comments
Assignees
@CreepPork
Labels
bug Issue that is not working as intended. priority/medium Issue or PR that impacts some portions of the user base and should be resolved with due time.
Milestone
1.0.0

Comments

@McDiod
Copy link

McDiod commented Dec 16, 2017 •
edited by CreepPork
Loading

AresModAchillesExpansion Version: Alpha-009c

Description:
When headlight status of a non-local vehicle is set, RscAttributeHeadlight calls Achilles_fnc_spawn for remote execution:

https://github.com/ArmaAchilles/AresModAchillesExpansion/blob/22111bcebe890f076a4ef37882534a380080fe82/%40AresModAchillesExpansion/addons/ui_f/scripts/RscAttributeHeadlight.sqf#L88

Achilles_fnc_spawn then uses remoteExecCall, creating a non-scheduled environment...

https://github.com/ArmaAchilles/AresModAchillesExpansion/blob/22111bcebe890f076a4ef37882534a380080fe82/%40AresModAchillesExpansion/addons/functions_f_achilles/functions/common/fn_spawn.sqf#L30

...where this part of RscAttributeHeadlight's _codeBlock spams the log with errors:

https://github.com/ArmaAchilles/AresModAchillesExpansion/blob/22111bcebe890f076a4ef37882534a380080fe82/%40AresModAchillesExpansion/addons/ui_f/scripts/RscAttributeHeadlight.sqf#L84

Steps to reproduce:
Set headlight status of non-local vehicle.

Where did the issue occur?

  • Dedicated

RPT log file:

20:39:42   Error position: <sleep 0.01;
_entity setpilotlight _light>
20:39:42   Error Generic error in expression
20:39:42 Suspending not allowed in this context
20:39:42 Error in expression <light","auto"]) isEqualTo _light)} do
{
sleep 0.01;
_entity setpilotlight _light>
@CreepPork CreepPork self-assigned this Dec 16, 2017
@CreepPork CreepPork added bug Issue that is not working as intended. priority/medium Issue or PR that impacts some portions of the user base and should be resolved with due time. labels Dec 16, 2017
@CreepPork CreepPork added this to the 0.1.0 milestone Dec 16, 2017
CreepPork added a commit that referenced this issue Dec 16, 2017
@CreepPork
Fixed #202
e807f11
CreepPork pushed a commit that referenced this issue Dec 23, 2017
@unhappytroll @CreepPork
Added Russian translation (#192)
b9607b9 
* Added Russian translation

Added Russian translation to all keys in @AresModAchillesExpansion\addons\language_f\stringtable.xml.
Polished existed translation a little.

* my chganges

* Fixed localization

Fixed localisation string for aborting fire mission message.

* Fixed some localisation issues

Maked different localisation strings for both modules.

* copied

* Added warning string

Added warning do dialog

* Added localization to dialog

Added localization to dialod code.

* Massive changes

- [Rollback] Removed changes in categories names due to selection logic;
- [Tweak] Reworked translation (too long strings)
- [New] Added some new containers for sorting sake
-[Tweak] Preliminary sorting of keys by containers;
- [New] Added new localisation keys for nonlocalized scripts

* reverted

* Error fixing

-[Fix] Fixed some errors in localization keys

* Changed one localized string for convenience sake

* Changed Alt Change dialog

Added localized strings.
Added warning message.

* Localized dialod

Localised dialog, fixed one string for uniformity

* Changed one string for convenience sake

* More chages

Added strings for newly localized dilogs
Russian translation polished further (shorted some long strings, which has trouble to fit into controls, dialogs and menu

* Fixed discrepancy in one german string as requested

* Added localization for error messages

* changed distance localization

* Changed distance localization

* Added error message localization

* Localization added

Added localization strings for errors and sides

* redone localization for less strings amount

* Multiple keys added, some strings fixed for length

* Made requested changes

* Made requested changes

* added some strings

* Changes according to request

* Mine detection ability added and localized

* Resolving conflicts

* Added localization

* added localization

* More fixes

More fixes, added new strings

* Adding localization strings

* Requested corrections

* Moved AddonBuilder.exe out of this repository and updated README.md

* Removed AddonBuilder.exe source

* Added Tac-Ops action event tracks #193

* Updated debug message

* Code optimization

* Supply Drop properly checks if no LZs have been found #197

* Intro hint will now only be shown once

* Removed misplaced icon file

* Nitpicky changes in stringtable.xml

* Added Arsenal Adding and removing modules #200

* Updated changelog

* Added shukari's contribution to translations

* Updated the issue template

* Fixed #202

* Execute Code module now saves last piece of code to profileNamespace

* Fixed #202

* Code refactoring

* Fixed typo

* Added Copy to Clipboard Arsenal module

* Fixed duplicate and a stray Key tag

* Create Teleporter module now takes the Cancel button correctly into account #203

* Updated changelog

* Changed to have more meaningful info displayed

* Added Combine and Replace inventory modules into one #200

* Added feedback message #200

* User now has ability to fire custom amount of rounds

* Fixed Custom Fire not spawning where desired

* Select Units default now should be localized

* Fixed Achilles creating dependencies #200

* Added Arsenal Create Custom module #200

* Added partial ACE Arsenal compatability #200

* Added option to select weapon and AI should aim correctly #188

* Updated changelog

* Grammar fixes

* Removed misleading comments

* Returns correct amount of added or removed units #170

* Added partial ACE 3 Arsenal compatability

* Renamed logMessage to log

* Updated changelog

* Fixed errors

* Added missing strings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@CreepPork CreepPork

Labels
bug Issue that is not working as intended. priority/medium Issue or PR that impacts some portions of the user base and should be resolved with due time.
Projects
None yet
Milestone
1.0.0
Development

No branches or pull requests
2 participants
@CreepPork @McDiod