| Index: trunk/extensions/CodeReview/backend/CodeRevision.php |
| — | — | @@ -778,10 +778,11 @@ |
| 779 | 779 | public function getComments( $attached = false ) { |
| 780 | 780 | $conditions = array( |
| 781 | 781 | 'cc_repo_id' => $this->repoId, |
| 782 | | - 'cc_rev_id' => $this->id ); |
| | 782 | + 'cc_rev_id' => $this->id |
| | 783 | + ); |
| 783 | 784 | |
| 784 | 785 | if( $attached ) { |
| 785 | | - $conditions['cc_patch_line'] = '!= null'; |
| | 786 | + $conditions[] = 'cc_patch_line != null'; |
| 786 | 787 | } else { |
| 787 | 788 | $conditions['cc_patch_line'] = null; |
| 788 | 789 | } |
| Index: trunk/extensions/CodeReview/backend/DiffHighlighter.php |
| — | — | @@ -13,9 +13,14 @@ |
| 14 | 14 | /* line number inside patch */ |
| 15 | 15 | protected $lineNumber = 0; |
| 16 | 16 | |
| 17 | | - /* CodeRepository: The repository for this revision */ |
| | 17 | + /** |
| | 18 | + * @var CodeRepositor The repository for this revision |
| | 19 | + */ |
| 18 | 20 | protected $repo = null; |
| 19 | | - /* CodeRevision: revision the diff comes from */ |
| | 21 | + |
| | 22 | + /** |
| | 23 | + * @var CodeRevision revision the diff comes from |
| | 24 | + */ |
| 20 | 25 | protected $rev = null; |
| 21 | 26 | |
| 22 | 27 | /** |