OFPSDEV / BECTI

Mission Development for OFPS
3 stars 4 forks source link

Satchels not attaching all vehicles #34

Open SpanishSurfer opened 10 years ago

SpanishSurfer commented 10 years ago

We have a satchel attach to vehicle script that works pretty decent, only problem is that it doesn't work with every vehicle.

Liveandletdie commented 10 years ago

Seems to only work on 1st life as well, like after you die once it no longer works....looking into it

Liveandletdie commented 10 years ago

Could be a nice alternative once MP support......http://www.armaholic.com/page.php?id=22058

Liveandletdie commented 10 years ago

added support for more vehicle types but still need to get it to work after you re spawn.....any ideas?

MarkusNemesis commented 10 years ago

I can look into it. Where is this script located?

On 22 September 2014 11:50, Liveandletdie notifications@github.com wrote:

added support for more vehicle types but still need to get it to work after you re spawn.....any ideas?

— Reply to this email directly or view it on GitHub https://github.com/OFPSDEV/BECTI/issues/34#issuecomment-56326357.

Liveandletdie commented 10 years ago

init.sqf line 198 calls the script script located in addons/EtV.sqf

MarkusNemesis commented 10 years ago

Resolved in commit 1e93961161542a87612e0caa594e655c23675fa4 . Please refer to the description on the commit.

Cheers,

SpanishSurfer commented 10 years ago

Could not reproduce working results. Tried on PANTHER, OFFROAD, AND BLUFOR MBT the script did not load for me. This was not done in the editor but through a dedicated server.

MarkusNemesis commented 10 years ago

Did you test with -showscripterrors enabled? Do you have any errors in your .RPT?

MarkusNemesis commented 10 years ago

Commit 7e23d099136e7adda951a7f63397a88e3ac2a0d7 should now have it resolved. If not, please provide any errors from -showscripterrors or .rpt errors so that I can investigate.

Cheers,

SpanishSurfer commented 10 years ago

I'll test tonight, I was able to attach satchels to the HEMTT. Seems like it was vehicle specific.

MarkusNemesis commented 10 years ago

Tested and appears to work correctly on all light vehicles and tanks.

The player needs to be on a certain location on each to ensure they're close enough to the centre point of the vehicle, to enable them to interact. 75170d0c6cd11dad3493602befe7c51fb4386071

SpanishSurfer commented 10 years ago

Tried this on mhq, HEMT, offroad, and UGV and it did not work for me. I walked around every vehicle with satchels in backpack, belt, and uniform. Did not work at all. Tested on a dedicated server.

MarkusNemesis commented 10 years ago

I tested on an offroad and it worked correctly. Are you scrolling low enough in your list to see it? It's quite low down there.

You say you were not able to attach to the HEMT, yet one comment ago you said you could?

Are you using the correct charges? Satchels don't count. This actually uses Explosive Charges as the munition.

On 26 September 2014 06:56, SpanishSurfer notifications@github.com wrote:

Tried this on mhq, HEMT, offroad, and UGV and it did not work for me. I walked around every vehicle with satchels in backpack, belt, and uniform. Did not work at all. Tested on a dedicated server.

— Reply to this email directly or view it on GitHub https://github.com/OFPSDEV/BECTI/issues/34#issuecomment-56896193.

SpanishSurfer commented 10 years ago

You're right I'm being confusing. I am using the explosive charges and I did say it worked before. However, when I tested it I was on the mission editor. I'm going all the way down the scroll menu, and nothing is coming up (we should increase the priority of that too 98). I'm on a dedicated server now and it's not working.

MarkusNemesis commented 10 years ago

Again, any script errors? rpt logs? etc?

MarkusNemesis commented 10 years ago

Tidied up the code a little in the EtV script as per 1d6ced2a2c0e9b7714f896f3168fd02777316bd5

I'll implement a better proximity detection function soon.

MarkusNemesis commented 10 years ago

efe91e3

Should no longer break spawning. Changed a function that was misused.

Liveandletdie commented 10 years ago

Big one, explosives do not hurt the vehicle it is attached to (maybe specific to UGV), also need to add dialog while within vehicles

SpanishSurfer commented 10 years ago

The script has been working (I've seen it used on Darters to blow up shit). Anyway we can limit how many satchels we allow to be placed on a darter?