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.

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

Can players be required as flight blocks. #163

Open
pistols12 opened this issue Mar 29, 2016 · 14 comments
Open

Can players be required as flight blocks. #163

pistols12 opened this issue Mar 29, 2016 · 14 comments

Comments

@pistols12
Copy link

Hello comma I'm wondering if it is possible to make players fly blocks. To basically a sign players their own unique ID, and I mean players in general not specific ones. I want to use this to say make a ram vehicle that requires multiple people before it can operate.

@mwkaicz
Copy link
Contributor

mwkaicz commented Mar 30, 2016

Hi, this is interesting idea.
At this point only action with players is allow/deny all entities to move with the craft.
Your request is easy for implementation, but I'm not sure how act with these lists of players.
Are you speaking about two futures or only one?

As I see there are two options here:

  1. Specify number of needed players on the pilot sing - easy way, just a number and check of players count on the board.
  2. make signifacation signs, where must be UUID/user name ... and here it will be one sign per user ... but this will be little harder, because there must be some rules when you can put your signature on the ship and when you can't.

It's not first time, when is this discussed, but still without results :/

P.S: please, create tickets only on https://github.com/msummers123/Movecraft-3/

@pistols12
Copy link
Author

The first option makes the most sense to me
On Mar 30, 2016 1:51 AM, "Kai" [email protected] wrote:

Hi, this is interesting idea.
At this point only action with players is allow/deny all entities to move
with the craft.
Your request is easy for implementation, but I'm not sure how act with
these lists of players.
Are you speaking about two futures or only one?

As I see there are two options here:

  1. Specify number of needed players on the pilot sing - easy way, just a
    number and check of players count on the board.
  2. make signifacation signs, where must be UUID/user name ... and here it
    will be one sign per user ... but this will be little harder, because there
    must be some rules when you can put your signature on the ship and when you
    can't.

It's not first time, when is this discussed, but still without results :/


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#163 (comment)

@pistols12
Copy link
Author

I figured it would be a simple implementation so really the simplest way to
do this is probably the best. I'm just looking to for example make my Ram
craft require two or more people. But I also see a lot of ways this could
be used. Especially for my medieval themed server.
On Mar 30, 2016 11:41 AM, "Logan Clayton" [email protected] wrote:

The first option makes the most sense to me
On Mar 30, 2016 1:51 AM, "Kai" [email protected] wrote:

Hi, this is interesting idea.
At this point only action with players is allow/deny all entities to move
with the craft.
Your request is easy for implementation, but I'm not sure how act with
these lists of players.
Are you speaking about two futures or only one?

As I see there are two options here:

  1. Specify number of needed players on the pilot sing - easy way, just a
    number and check of players count on the board.
  2. make signifacation signs, where must be UUID/user name ... and here it
    will be one sign per user ... but this will be little harder, because there
    must be some rules when you can put your signature on the ship and when you
    can't.

It's not first time, when is this discussed, but still without results :/


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#163 (comment)

@pistols12
Copy link
Author

I'm in the process of learning how to code in Java, as well as do the
Bukkit API. I am far from making my own plug-ins. But eventually I bet I
would be able to do this by myself, I was just hoping someone who knows a
bit more would be able to make a quick change an update movecraft.
On Mar 30, 2016 11:42 AM, [email protected] wrote:

I figured it would be a simple implementation so really the simplest way
to do this is probably the best. I'm just looking to for example make my
Ram craft require two or more people. But I also see a lot of ways this
could be used. Especially for my medieval themed server.
On Mar 30, 2016 11:41 AM, "Logan Clayton" [email protected] wrote:

The first option makes the most sense to me
On Mar 30, 2016 1:51 AM, "Kai" [email protected] wrote:

Hi, this is interesting idea.
At this point only action with players is allow/deny all entities to
move with the craft.
Your request is easy for implementation, but I'm not sure how act with
these lists of players.
Are you speaking about two futures or only one?

As I see there are two options here:

  1. Specify number of needed players on the pilot sing - easy way, just a
    number and check of players count on the board.
  2. make signifacation signs, where must be UUID/user name ... and here
    it will be one sign per user ... but this will be little harder, because
    there must be some rules when you can put your signature on the ship and
    when you can't.

It's not first time, when is this discussed, but still without results :/


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#163 (comment)

@pistols12
Copy link
Author

I want to code specifically because I have very very interesting ideas that
are not done yet. I do not want to elaborate on them too much because I
plan on making a server around these unique ideas.
On Mar 30, 2016 11:44 AM, "Logan Clayton" [email protected] wrote:

I'm in the process of learning how to code in Java, as well as do the
Bukkit API. I am far from making my own plug-ins. But eventually I bet I
would be able to do this by myself, I was just hoping someone who knows a
bit more would be able to make a quick change an update movecraft.
On Mar 30, 2016 11:42 AM, [email protected] wrote:

I figured it would be a simple implementation so really the simplest way
to do this is probably the best. I'm just looking to for example make my
Ram craft require two or more people. But I also see a lot of ways this
could be used. Especially for my medieval themed server.
On Mar 30, 2016 11:41 AM, "Logan Clayton" [email protected]
wrote:

The first option makes the most sense to me
On Mar 30, 2016 1:51 AM, "Kai" [email protected] wrote:

Hi, this is interesting idea.
At this point only action with players is allow/deny all entities to
move with the craft.
Your request is easy for implementation, but I'm not sure how act with
these lists of players.
Are you speaking about two futures or only one?

As I see there are two options here:

  1. Specify number of needed players on the pilot sing - easy way, just
    a number and check of players count on the board.
  2. make signifacation signs, where must be UUID/user name ... and here
    it will be one sign per user ... but this will be little harder, because
    there must be some rules when you can put your signature on the ship and
    when you can't.

It's not first time, when is this discussed, but still without results
:/


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#163 (comment)

@mwkaicz
Copy link
Contributor

mwkaicz commented Mar 31, 2016

It will be not hard, but some changes will be needed.

  1. It can't be directly as flyBlocks (here it will make more harm than good), but we can add something like requireCrewSize: 3 in to the .craft file
  2. first what you must change craft/craftType.java for adding new property, that's the easiest part
  3. then you must create validation for players count (in Utils it will be fine) and add calling it to the async/translation/TranslationTask.java and async/rotation/RotationTask.java ... here you can use getCraft().getType().getMoveEntities() for get all entities on the craft, select only players and compare it with your new property getCraft().getType().getRequireCrewSize()
    ... that's (should be) all folks ... dadadada :)

@AJCStriker
Copy link
Owner

Assuming not much has changed in 5 years ( Jesus christ ) then the core of Movecraft 3 should still have a generic detection algorithm for all entities on board which is used to detect players for teleportation.

You could implement this feature quite easily and computationally efficiently with that algorithm.

@pistols12
Copy link
Author

Thanks guys, I'll have to see if i can't get this idea working! 😀 hot
damn! It's like Christmas in july.... But it's March. 😆
On Mar 31, 2016 4:36 AM, "Alexander Christie" [email protected]
wrote:

Assuming not much has changed in 5 years ( Jesus christ ) then the core of
Movecraft 3 should still have a generic detection algorithm for all
entities on board which is used to detect players for teleportation.

You could implement this feature quite easily and computationally
efficiently with that algorithm.


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#163 (comment)

@pistols12
Copy link
Author

Still kinda hazey on AJC strikers response, how would I do that?

And have not attempted kai's method as honestly the steps dont make 100 percent sense to me, i get the framework but not what to actually change and where.

@mwkaicz
Copy link
Contributor

mwkaicz commented May 18, 2016

I will put it into my next merge request (during next two weeks, I haven't too much free time now). Be patient.

@pistols12
Copy link
Author

Thanks, no hurry on my end
On May 17, 2016 11:15 PM, "Kai" [email protected] wrote:

I will put it into my next merge request (during next two weeks, I haven't
too much free time now). Be patient.


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#163 (comment)

@pistols12
Copy link
Author

did this get implemented?

@mwkaicz
Copy link
Contributor

mwkaicz commented Aug 12, 2016

Not jet, sorry, there was little bit chaos about merges. I'm waiting until situation will be calmer.

@pistols12
Copy link
Author

Added Yet?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants