MurderMysteryPE icon indicating copy to clipboard operation
MurderMysteryPE copied to clipboard

Project Discussion

Open corytortoise opened this issue 8 years ago • 21 comments

Project Discussion

Let's use this issue to discuss anything project-related, including quick questions, contribution discussion, etc. For other issues, please make a separate issue in the tracker to prevent confusion.

corytortoise avatar Apr 28 '17 13:04 corytortoise

Considering the amount of people that want to contribute, I believe we need to make a consensus on contribution guidelines. I suggest we use the same code formatting as PMMP, but I am open to any suggestions.

corytortoise avatar Apr 28 '17 13:04 corytortoise

This plugin is going to be a lot bigger a deal than I had anticipated

jasonw4331 avatar Apr 28 '17 14:04 jasonw4331

Can i put my own version here? Just to test it out and improves it?

ghost avatar Apr 28 '17 14:04 ghost

@imjayl you already have a version that works?

jasonw4331 avatar Apr 28 '17 14:04 jasonw4331

@jasonwynn10 Yes its easy to customize but the only problem is i didnt add the gold spawn thing

ghost avatar Apr 28 '17 14:04 ghost

link?

jasonw4331 avatar Apr 28 '17 14:04 jasonw4331

View your message at pmmp forum :)

ghost avatar Apr 28 '17 14:04 ghost

@jasonwynn10 can you create a proper issue with the total list of features?

xBeastMode avatar Apr 28 '17 16:04 xBeastMode

Put @imjayl code into separate branch.

Kris-Driv avatar Apr 28 '17 16:04 Kris-Driv

He asks that the code is not to be made public, so we won't be able to copy it explicitly, but I think we may be able to use it as an example.

jasonw4331 avatar Apr 28 '17 16:04 jasonw4331

Also, make sure that we don't make too much code changes per commit. In project like this and at this stage, it can cause file conflicts a lot.

Kris-Driv avatar Apr 28 '17 16:04 Kris-Driv

@Chris-Prime I agree.

corytortoise avatar Apr 28 '17 18:04 corytortoise

And if it's possible please use Git, it will automatically sync the repository for you, then you can do your part and commit it.

xBeastMode avatar Apr 28 '17 19:04 xBeastMode

I won't be able to really code very much until I can get my computer repaired, but I will be using PhpStorm once I do, so you all shouldn't have anything to worry about.

jasonw4331 avatar Apr 28 '17 19:04 jasonw4331

Main class name seems too plain, what about Launcher or MurderMystery?

Kris-Driv avatar Apr 28 '17 20:04 Kris-Driv

MurderMystery is good.

xBeastMode avatar Apr 28 '17 20:04 xBeastMode

At some point we'll need to implement language files. What's your opinion on using namespace function which would look like this

namespace TurtleTeam\MurderMystery;

function lang(string $key, ...$params): string {
   return Foo::getInstance()->getMessage($key, $params);
}

class Foo {
   
   public function onEnable() {
      $this->getLogger()->info(lang("plugin-enabling"));
   }
}

Kris-Driv avatar Apr 28 '17 20:04 Kris-Driv

I guess it's good but I think we should make it static final just for security.

xBeastMode avatar Apr 28 '17 20:04 xBeastMode

People, is this plugin still going to be continued? I can do all the bodies part and stuff if you want.

Sandertv avatar Jul 09 '17 21:07 Sandertv

I have my own project running separately which has made more progress... I'm pretty sure this is discontinued, but if the others feel like working on it, they may feel free to do so

jasonw4331 avatar Jul 10 '17 01:07 jasonw4331

If somebody wants to take over ownership, let me know. I'm not worthy or or knowledgable enough to oversee this.

corytortoise avatar Jul 14 '17 17:07 corytortoise