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

Documentation for each function and MD documentation #10

Open
PEMapModder opened this issue Jul 16, 2014 · 35 comments
Open

Documentation for each function and MD documentation #10

PEMapModder opened this issue Jul 16, 2014 · 35 comments
Assignees

Comments

@PEMapModder
Copy link
Member

Write documentation in the following syntax:

/**
 * Method details and elaboration in HTML
 * @param ParamType $paramName elaboration
 * @return ReturnType elaboration
*/

Do it for every API function (that is supposed that other plugins can use).

@PEMapModder
Copy link
Member Author

Assign yourself as "Assignee" if you want to do this.

@PEMapModder PEMapModder changed the title Documentation for each function Documentation for each function and MD documentation Jul 18, 2014
@PEMapModder
Copy link
Member Author

@iJoshuaHD is assigned to #10.

@iJoshuaHD
Copy link

@PEMapModder sorry but i dont know

/**

  • Method details and elaboration in HTML
  • @param ParamType $paramName elaboration
  • @return ReturnType elaboration
    */

stands for? or it just explains how this function works?

i thought i will be working on Documentation xD

@PEMapModder
Copy link
Member Author

You can just work on the documentation part :)

@iJoshuaHD
Copy link

pheww *

@PEMapModder
Copy link
Member Author

Lol there is really someone called @return. Must get a lot of notifications ;)
@param too

@PEMapModder
Copy link
Member Author

It explains what the function is for. Only API functions need this, though, since internal functions don't need to get documented.

@iJoshuaHD
Copy link

yea i dont understand how the documentation thingy works xD

@PEMapModder
Copy link
Member Author

Just do the owner guide and player guide then.

Documentation comment is a format. Basically it just explains what the function is for.

@iJoshuaHD
Copy link

will do :)

@alejzeis
Copy link

Dis is called java doc.

@PEMapModder
Copy link
Member Author

PHP doc it is called in PHPStorm.

@alejzeis
Copy link

It's basically a copy of java doc...

On Tuesday, July 22, 2014, PEMapModder [email protected] wrote:

PHP doc it is called in PHPStorm.


Reply to this email directly or view it on GitHub
#10 (comment)
.

@PEMapModder
Copy link
Member Author

I don't think the | thing is copied.

@alejzeis
Copy link

Where's that?

On Tuesday, July 22, 2014, PEMapModder [email protected] wrote:

I don't think the | thing is copied.


Reply to this email directly or view it on GitHub
#10 (comment)
.

@PEMapModder
Copy link
Member Author

@jython234

/**
 * @param string|Player $player
 * @return string
 */
public function getPlayerFile($player){
    if($player instanceof Player) $player = $player->getName();
    return $this->getDataFolder().strtolower($player).".yml";
}

@swagking47
Copy link

@PEMapModder PEMapModder does every player have his own file ?

@alejzeis
Copy link

I guess your right, but still those tags and the doc format look like java
doc to me...

On Tuesday, July 22, 2014, swagking47 [email protected] wrote:

PEMapModder does every player have his own file ?


Reply to this email directly or view it on GitHub
#10 (comment)
.

@PEMapModder
Copy link
Member Author

@swagking47 just an example.

@caingaroo
Copy link

Thee plugin has a problem in the Faction ranks
[Server] 19:33:13 [INFO] Loading DevTools v1.10.0
[Server] 19:33:18 [INFO] Enabling DevTools v1.10.0
[Server] 19:33:18 [INFO] Loading source plugin PocketFactions v2.0.0
[Server] 19:33:18 [INFO] [DevTools] Registered folder plugin loader
[Server] 19:33:18 [INFO] Preparing level "Arena"
[Server] 19:33:18 [INFO] Enabling PocketFactions v2.0.0
[Server] 19:33:18 [NOTICE] ArrayOutOfBoundsException: "Undefined index: permissions" (E_NOTICE) in "/PocketFactions-new/src/pocketfactions/faction/FactionRank" at line 32
[Server] 19:33:18 [INFO] Disabling PocketFactions v2.0.0
[Server] 19:33:19 [INFO] Starting GS4 status listener
[Server] 19:33:19 [INFO] Setting query port to 19132
[Server] 19:33:19 [INFO] Query running on 0.0.0.0:19132
[Server] 19:33:19 [INFO] Default game type: SURVIVAL
[Server] 19:33:19 [INFO] Done (13.005s)! For help, type "help" or "?"

@PEMapModder
Copy link
Member Author

@caingaroo plugin is not finished yet.

@caingaroo
Copy link

@PEMapModders why didnt you labble it on the plugin , or did i miss it somehoe 😯😯

@PEMapModder
Copy link
Member Author

If the plugin is completed, we would make a release.
Where do you think we would label it?

@caingaroo
Copy link

Dont know, but could you notify me win the plugin is finished

@alejzeis
Copy link

Ehh doxygen.

@PEMapModder
Copy link
Member Author

We will create a release when a beta is ready, and submit on forums when initial release is ready.

@PEMapModder
Copy link
Member Author

@jython234 you don't need to compile it. It can load source plugin.

@PEMapModder
Copy link
Member Author

@astersx how are they related?

@iksaku
Copy link

iksaku commented Jan 2, 2016

They are not related at all..

@PEMapModder
Copy link
Member Author

@astersx

  1. It is supposed to support as many economy plugins as possible, although @onebone's economys and @LegendOfMCPE's xEcon (W.I.P.) will have priority.
  2. If CustomRanks overrides other plugins' tags, it has a problem. Don't use it.
  3. Why do you even use Economys Land with Factions? That's unreasonable.
  4. DIsable commands? Just revoke the permissions with a permissions plugin.
  5. If you need to avoid making multiple accounts, maybe you should use an auth plugin that checks that, e.g. ServerAuth by EvolSoft.

@PEMapModder
Copy link
Member Author

What is this? https://github.com/EvolSoft/ServerAuth/blob/master/ServerAuth/resources/config.yml#L30

Also, please don't use the issue tracker as a chatroom. Get on our IRC channel or Gitter chatroom if you need further support.

@LegendOfMCPE LegendOfMCPE locked and limited conversation to collaborators Jan 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants
@iksaku @PEMapModder @caingaroo @alejzeis @iJoshuaHD @swagking47 and others