mmainstreet /
jodel-web
These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more
| 1 | <?php |
||
| 2 | error_reporting(-1); |
||
| 3 | |||
| 4 | include 'php/DatabaseConnect.php'; |
||
| 5 | include 'php/Requests/AbstractRequest.php'; |
||
| 6 | include 'php/Requests/CreateUser.php'; |
||
| 7 | include 'php/AccountData.php'; |
||
| 8 | include 'php/Location.php'; |
||
| 9 | include 'php/Requests/GetPosts.php'; |
||
| 10 | include 'php/Requests/GetKarma.php'; |
||
| 11 | include 'php/Requests/UpdateLocation.php'; |
||
| 12 | include 'php/Requests/Upvote.php'; |
||
| 13 | include 'php/Requests/Downvote.php'; |
||
| 14 | include 'php/Requests/GetPostDetails.php'; |
||
| 15 | include 'php/Requests/SendJodel.php'; |
||
| 16 | |||
| 17 | require_once 'php/Requests/libary/Requests.php'; |
||
| 18 | Requests::register_autoloader(); |
||
| 19 | |||
| 20 | function getPosts($lastPostId, $url) { |
||
| 21 | $db = new DatabaseConnect(); |
||
| 22 | if ($db->connect_errno) { |
||
| 23 | echo 'Sorry, die Verbindung zu unserem superfetten endgeilen |
||
| 24 | Server ist hops gegangen. Wegen '. $db -> connect_error; |
||
| 25 | } |
||
| 26 | |||
| 27 | $result = $db->query("SELECT * FROM accounts WHERE id='1'"); |
||
| 28 | |||
| 29 | $access_token; |
||
|
0 ignored issues
–
show
|
|||
| 30 | |||
| 31 | if ($result->num_rows > 0) { |
||
| 32 | // output data of each row |
||
| 33 | while($row = $result->fetch_assoc()) { |
||
| 34 | $access_token = $row["access_token"]; |
||
| 35 | } |
||
| 36 | } else { |
||
| 37 | echo "0 results"; |
||
| 38 | } |
||
| 39 | |||
| 40 | $accountCreator = new GetPosts(); |
||
| 41 | $accountCreator->setLastPostID($lastPostId); |
||
| 42 | $accountCreator->setUrl($url); |
||
| 43 | $accountCreator->setAccessToken($access_token); |
||
| 44 | $data = $accountCreator->execute(); |
||
| 45 | |||
| 46 | return $data; |
||
| 47 | } |
||
| 48 | $posts; |
||
| 49 | |||
| 50 | View Code Duplication | if(isset($_GET['commentView'])) |
|
| 51 | { |
||
| 52 | $commentView = true; |
||
| 53 | $url = "/posts/location/discussed/"; |
||
| 54 | } |
||
| 55 | else |
||
| 56 | { |
||
| 57 | if(isset($_GET['upVoteView'])) |
||
| 58 | { |
||
| 59 | $upVoteView = true; |
||
| 60 | $url = "/posts/location/popular/"; |
||
| 61 | } |
||
| 62 | else |
||
| 63 | { |
||
| 64 | $timeView = true; |
||
| 65 | $url = "/v2/posts"; |
||
| 66 | } |
||
| 67 | } |
||
| 68 | |||
| 69 | if(isset($_GET['lastPostId'])) { |
||
| 70 | |||
| 71 | $lastPostId = $_GET['lastPostId']; |
||
| 72 | |||
| 73 | $posts = getPosts($lastPostId, $url)['posts']; |
||
| 74 | $loops = 29; |
||
| 75 | $showCommentIcon = TRUE; |
||
| 76 | ?> |
||
| 77 | <div class="nextPosts"> |
||
| 78 | <?php |
||
| 79 | for($i = 0; $i<$loops; $i++) { |
||
| 80 | |||
| 81 | if(isset($posts[$i])) { |
||
| 82 | $lastPostId = $posts[$i]['post_id']; |
||
| 83 | |||
| 84 | |||
| 85 | $now = new DateTime(); |
||
| 86 | $d = new DateTime($posts[$i]["created_at"]); |
||
| 87 | |||
| 88 | |||
| 89 | //Time to time difference |
||
| 90 | $timediff = $now->diff($d); |
||
| 91 | |||
| 92 | $timediff_inSeconds = (string)$timediff->format('%s'); |
||
| 93 | $timediff_inMinutes = (string)$timediff->format('%i'); |
||
| 94 | $timediff_inHours = (string)$timediff->format('%h'); |
||
| 95 | $timediff_inDays = (string)$timediff->format('%d'); |
||
| 96 | $timediff_inMonth = (string)$timediff->format('%m'); |
||
| 97 | View Code Duplication | if($timediff_inMonth!=0) { |
|
| 98 | $timediff = $timediff_inMonth . "m"; |
||
| 99 | } |
||
| 100 | else |
||
| 101 | { |
||
| 102 | if($timediff_inDays!=0) |
||
| 103 | { |
||
| 104 | $timediff = $timediff_inDays . "d"; |
||
| 105 | } |
||
| 106 | else |
||
| 107 | { |
||
| 108 | if($timediff_inHours!=0) |
||
| 109 | { |
||
| 110 | $timediff = $timediff_inHours . "h"; |
||
| 111 | } |
||
| 112 | else |
||
| 113 | { |
||
| 114 | if($timediff_inMinutes!=0) |
||
| 115 | { |
||
| 116 | $timediff = $timediff_inMinutes . "m"; |
||
| 117 | } |
||
| 118 | else |
||
| 119 | { |
||
| 120 | $timediff = $timediff_inSeconds . "s"; |
||
| 121 | } |
||
| 122 | } |
||
| 123 | } |
||
| 124 | } |
||
| 125 | ?> |
||
| 126 | |||
| 127 | <article class="jodel" style="background-color: #<?php echo $posts[$i]["color"];?>;"> |
||
| 128 | <content> |
||
| 129 | <?php |
||
| 130 | View Code Duplication | if(isset($posts[$i]["image_url"])) { |
|
| 131 | echo '<img src="' . $posts[$i]["image_url"] . '">'; |
||
| 132 | } |
||
| 133 | else { |
||
| 134 | echo nl2br($posts[$i]["message"]); |
||
| 135 | } |
||
| 136 | ?> |
||
| 137 | </content> |
||
| 138 | <aside> |
||
| 139 | <a href="index.php?vote=up&postID=<?php echo $posts[$i]["post_id"];?>"> |
||
| 140 | <i class="fa fa-angle-up fa-3x"></i> |
||
| 141 | </a> |
||
| 142 | <br /> |
||
| 143 | <?php echo $posts[$i]["vote_count"];?><br /> |
||
| 144 | <a href="index.php?vote=down&postID=<?php echo $posts[$i]["post_id"];?>"> |
||
| 145 | <i class="fa fa-angle-down fa-3x"></i> |
||
| 146 | </a> |
||
| 147 | </aside> |
||
| 148 | |||
| 149 | <footer> |
||
| 150 | <table> |
||
| 151 | <tr> |
||
| 152 | <td class="time"> |
||
| 153 | <span data-tooltip="Time"> |
||
| 154 | <i class="fa fa-clock-o"></i> |
||
| 155 | <?php echo $timediff;?> |
||
| 156 | </span> |
||
| 157 | </td> |
||
| 158 | <td class="comments"> |
||
| 159 | View Code Duplication | <?php if($showCommentIcon) {?> |
|
| 160 | <span data-tooltip="Comments"> |
||
| 161 | <a href="index.php?getPostDetails=true&postID=<?php echo $posts[$i]["post_id"];?>"> |
||
| 162 | <i class="fa fa-commenting-o"></i> |
||
| 163 | <?php if(array_key_exists("child_count", $posts[$i])) { |
||
| 164 | echo $posts[$i]["child_count"]; |
||
| 165 | } else echo "0"; |
||
| 166 | ?> |
||
| 167 | </a> |
||
| 168 | </span> |
||
| 169 | <?php } ?> |
||
| 170 | </td> |
||
| 171 | <td class="distance"> |
||
| 172 | <span data-tooltip="Distance"> |
||
| 173 | <i class="fa fa-map-marker"></i> |
||
| 174 | <?php echo $posts[$i]["distance"];?> km |
||
| 175 | </span> |
||
| 176 | </td> |
||
| 177 | </tr> |
||
| 178 | </table> |
||
| 179 | </footer> |
||
| 180 | </article> |
||
| 181 | |||
| 182 | |||
| 183 | |||
| 184 | <?php |
||
| 185 | } |
||
| 186 | } |
||
| 187 | ?> |
||
| 188 | </div> |
||
| 189 | <div class="lastPostId"> |
||
| 190 | <?php echo $lastPostId; ?> |
||
| 191 | </div> |
||
| 192 | <?php |
||
| 193 | } |
||
| 194 |
This error can happen if you refactor code and forget to move the variable initialization.
Let’s take a look at a simple example:
The above code is perfectly fine. Now imagine that we re-order the statements:
In that case,
$xwould be read before it is initialized. This was a very basic example, however the principle is the same for the found issue.