Completed
Pull Request — gcconnex (#1517)
by Nick
22:12
created
mod/missions/api/v0/export.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -13,6 +13,9 @@
 block discarded – undo
13 13
 	private $before = null;
14 14
 	private $limit = null;
15 15
 
16
+	/**
17
+	 * @param string $object_type
18
+	 */
16 19
 	public function __construct(
17 20
 		$object_type,
18 21
 		$subtype = false,
Please login to merge, or discard this patch.
mod/missions/lib/missions-errors.php 1 patch
Doc Comments   +9 added lines patch added patch discarded remove patch
@@ -41,6 +41,9 @@  discard block
 block discarded – undo
41 41
  * Wes7ley
42 42
  * Wesley9
43 43
  */
44
+/**
45
+ * @param string $name
46
+ */
44 47
 function mm_is_valid_person_name($name)
45 48
 {
46 49
 	$regex = "/^[^0-9]+$/";
@@ -57,6 +60,9 @@  discard block
 block discarded – undo
57 60
  * 532K351
58 61
  * @578(532)
59 62
  */
63
+/**
64
+ * @param string $num
65
+ */
60 66
 function mm_is_guid_number($num)
61 67
 {
62 68
 	$regex = "/^[0-9]*$/";
@@ -112,6 +118,9 @@  discard block
 block discarded – undo
112 118
  * Tests a the start time and duration of the given day of the week.
113 119
  * Also defaults the minutes to '00' if the hour is set and the minute value is null.
114 120
  */
121
+/**
122
+ * @param string $day
123
+ */
115 124
 function mm_validate_time($day, $input_array)
116 125
 {
117 126
 	$err = '';
Please login to merge, or discard this patch.
mod/missions/lib/missions.php 1 patch
Doc Comments   +27 added lines patch added patch discarded remove patch
@@ -10,6 +10,12 @@  discard block
 block discarded – undo
10 10
 /*
11 11
  * Packs the language variables for a single language into a single string.
12 12
  */
13
+/**
14
+ * @param string|null $lwc
15
+ * @param string|null $lwe
16
+ * @param string|null $lop
17
+ * @param string|null $lang
18
+ */
13 19
 function mm_pack_language($lwc, $lwe, $lop, $lang)
14 20
 {
15 21
 	$returner = '';
@@ -41,6 +47,9 @@  discard block
 block discarded – undo
41 47
 /*
42 48
  * Unpacks a language string into an array of its component variable.
43 49
  */
50
+/**
51
+ * @param string|null $lang
52
+ */
44 53
 function mm_unpack_language($data_string, $lang)
45 54
 {
46 55
 	$returner = array();
@@ -71,6 +80,11 @@  discard block
 block discarded – undo
71 80
 /*
72 81
  * Packs the time (hour and minute) into a single string.
73 82
  */
83
+/**
84
+ * @param string|null $hour
85
+ * @param string|null $min
86
+ * @param string|null $day
87
+ */
74 88
 function mm_pack_time($hour, $min, $day)
75 89
 {
76 90
 	$returner = '';
@@ -95,6 +109,9 @@  discard block
 block discarded – undo
95 109
 /*
96 110
  * Unpacks the time into an array of hour and minute.
97 111
  */
112
+/**
113
+ * @param string|null $day
114
+ */
98 115
 function mm_unpack_time($data_string, $day)
99 116
 {
100 117
 	$returner = array();
@@ -427,6 +444,10 @@  discard block
 block discarded – undo
427 444
  * Notify user function which uses messages_send() to send on site notifications and mail() to send e-mail.
428 445
  * This is here because notify_user() is currently not working.
429 446
  */
447
+/**
448
+ * @param string $title_en
449
+ * @param string $title_fr
450
+ */
430 451
 function mm_notify_user($recipient, $sender, $subject, $title_en, $title_fr, $body_en, $body_fr)
431 452
 {
432 453
 	$recipient_entity = get_entity($recipient);
@@ -611,6 +632,9 @@  discard block
 block discarded – undo
611 632
 /*
612 633
  * Returns an accept button to a user who has received an offer if the mission is not full.
613 634
  */
635
+/**
636
+ * @param ElggUser|null $applicant
637
+ */
614 638
 function mm_finalize_button($mission, $applicant)
615 639
 {
616 640
 	$relationship_count = elgg_get_entities_from_relationship(array(
@@ -780,6 +804,9 @@  discard block
 block discarded – undo
780 804
 /*
781 805
  * Check if the user is opted in to any of the opt-in options.
782 806
  */
807
+/**
808
+ * @param ElggEntity|null $current_user
809
+ */
783 810
 function check_if_opted_in($current_user)
784 811
 {
785 812
 	//Nick - adding additional checks for other opportunity types
Please login to merge, or discard this patch.