[SOLVED] Collision pick up

Help & Support
41

I created a new script called dropable and deleted the collision and replaced it with the mouse event
however when I pasted in the coding for the actual entity to drop to the root file it did not work and event in the code editor was crossed off

42

Yes.

Below (outside) your onCollisionStart function.

I think it’s better to add it to the pickable script, but if you create a new script be sure everything in the script has the new script name.

43 
var Pickable = pc.createScript('pickable');

// initialize code called once per entity
Pickable.prototype.initialize = function () {
this.app.mouse.on(pc.EVENT_MOUSEDOWN, this.onMouseDown, this);
this.pickedUp = false;
this.entity.collision.on('collisionstart', this.onCollisionStart, this);
Pickable.prototype.onMouseDown = function (event) {
    if (event.button === pc.MOUSEBUTTON_RIGHT) {  
        this.entity.reparent(this.app.root.findByName('Root'));
        this.pickedUp = false;
    }
};

Pickable.prototype.onCollisionStart = function (result) {
    if (result.other.name === 'Player' && !this.pickedUp) {
        this.pickedUp = true;
        var point = result.other.findByName('Point');
        this.entity.reparent(point);
        this.entity.setPosition(point.getPosition());
    }
}

is this correct?

44

No, this part is inside the initialize function right now, that’s not correct.

45

I need this to be writen out please I cant put them in the right spot I have already tried so many tutorials

46

This is the whole onMouseDown function. There is already a onCollisionStart function in your script. Just copy and past this onMouseDown function below the other onCollisionStart function.

47 

// initialize code called once per entity
Pickable.prototype.initialize = function () {
this.pickedUp = false;
this.entity.collision.on('collisionstart', this.onCollisionStart, this);
Pickable.prototype.onMouseDown = function (event) {
    if (event.button === pc.MOUSEBUTTON_RIGHT) {  
        this.entity.reparent(this.app.root.findByName('Root'));
        this.pickedUp = false;
    }
};
};

Pickable.prototype.onCollisionStart = function (result) {
    if (result.other.name === 'Player' && !this.pickedUp) {
        this.pickedUp = true;
        var point = result.other.findByName('Point');
        this.entity.reparent(point);
        this.entity.setPosition(point.getPosition());
    }

};
48

if this does not work does the script go under the pickable.prototype.oncollisionstart = function (result)

49

Why you don’t try to do, what I explain? First of all, remove the onMouseDown from the initialize function.

50

done back with the normal pickable script

51

I give up. Here you go:

var Pickable = pc.createScript('pickable');

// initialize code called once per entity
Pickable.prototype.initialize = function () {
    this.pickedUp = false;
    this.entity.collision.on('collisionstart', this.onCollisionStart, this);
    this.app.mouse.on(pc.EVENT_MOUSEDOWN, this.onMouseDown, this);
};

Pickable.prototype.onCollisionStart = function (result) {
    if (result.other.name === 'Player' && !this.pickedUp) {
        this.pickedUp = true;
        var point = result.other.findByName('Point');
        this.entity.reparent(point);
        this.entity.setPosition(point.getPosition());
    }
};

Pickable.prototype.onMouseDown = function (event) {
    if (event.button === pc.MOUSEBUTTON_RIGHT) {  
        this.entity.reparent(this.app.root.findByName('Root'));
        this.pickedUp = false;
    }
};
52

I am still a beginner.
sorry for wasting your time.

53

But still, your a great programmer I could have never gotten this far without you.

54

Thank You

55

I understand you are a beginner, that’s why I explain every step, you just have to follow the steps. If I do everything for you will not learn from it.

If something doesn’t work or isn’t right, restore it to the old situation, otherwise everything will be a mess.

56

Ok thank you.

57

Does it work right now?

58

Yes it did
it did not work with root as the file
I created a new entity and made it go there and now the script works perfectly.

59

This should work too, but I’m happy it works for you now.

60

Thanks.