jira-api-restclient
jira-api-restclient copied to clipboard
Feature/jira transitionbyname
Here 2 PRs are mixed into one:
-
deleteIssue
method -
transitionByStepName
method
I put it on top of my changes for jira-deleteIssue as i gave the other one to merge yesterday, if somebody really want to merge it i can prepare branch with only transitionbystepname function i would have few more updates probably soon, and would be hard to remember which functions are working with others.
This library seems to be most promising JIRA REST API client on PHP I've seen, but I'm not sure if it's maintained, because last commit was 1 year ago and there are lots of unmerged PRs.
Library creator however is still doing something on GitHub according to his feed.
I'm using it for few days - so don't know - it is very basic one but very easy to add your own functions. there is many jira api missing, but using the functions that u have here it's easy to build your own functions for jira API - probably the owner of the repo just make it for himself and just include functions he need - but it's good start - i send to the creator email about pull request - will see maybe there will be some updates
From architecture viewpoint there are some even more promising libraries, but they have 1 or 2 functions implemented, so I've chosen this library.
PR looks good to me. Splitting would be nice, also added some inline comments around spelling / indentation etc. Not sure if author is up for fixing these things else we could merge it anyways.
@jpastoor please add comments on PR's diff, rather then on commits in it because comments on commits are lost when squashing happens.
The https://github.com/chobie/jira-api-restclient/pull/57#issuecomment-152805635 isn't addressed as it seems.
StepName isn't a jira term, but it's easy to use step name (the name of the button above issue) to navigate. It's just easier to reffer it as text than as ID - even if text could be changed and ID cannot. but it would be more understandable if you would do: $jira->updatejira(...); $jira->transitionByStepName('Approved'); $jira->updatejira(..); $jira->transitionByStepName('Completed');
If you find it better (less going back and forward) I'm happy for you or anyone to make it properly :)
Please rebase and solve conflicts.