From 6cb7ffcd3baae236ad128d3745bbd3b7fd8a3ee5 Mon Sep 17 00:00:00 2001
From: Richard van der Hoff <richard@matrix.org>
Date: Sun, 18 Sep 2016 23:56:55 +0100
Subject: [PATCH] Another go at fixing the damn changelog

html_url is in the top-level 'commit' object, not in commit.commit.

Also factor some code out for legibility. Seriously, what's with stuffing
everything into inline functions?
---
 src/components/views/dialogs/ChangelogDialog.js | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/src/components/views/dialogs/ChangelogDialog.js b/src/components/views/dialogs/ChangelogDialog.js
index 6e1be907..e4a85f61 100644
--- a/src/components/views/dialogs/ChangelogDialog.js
+++ b/src/components/views/dialogs/ChangelogDialog.js
@@ -41,6 +41,16 @@ export default class ChangelogDialog extends React.Component {
         }
     }
 
+    _elementsForCommit(commit) {
+        return (
+            <li key={commit.sha} className="mx_ChangelogDialog_li">
+                <a href={commit.html_url} target="_blank" ref="noopener">
+                    {commit.commit.message}
+                </a>
+            </li>
+        );
+    }
+
     render() {
         const Spinner = sdk.getComponent('views.elements.Spinner');
         const QuestionDialog = sdk.getComponent('dialogs.QuestionDialog');
@@ -51,9 +61,7 @@ export default class ChangelogDialog extends React.Component {
                 <div key={repo}>
                     <h2>{repo}</h2>
                     <ul>
-                    {this.state[repo].map(commit =>
-                        <li key={commit.commit.url} className="mx_ChangelogDialog_li"><a href={commit.commit.html_url} target="_blank" ref="noopener">{commit.commit.message}</a></li>
-                    )}
+                    {this.state[repo].map(this._elementsForCommit)}
                     </ul>
                 </div>
             )