You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@whimsical.apache.org by se...@apache.org on 2020/07/08 23:22:07 UTC

[whimsy] branch master updated: Define extra before first use

This is an automated email from the ASF dual-hosted git repository.

sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/whimsy.git


The following commit(s) were added to refs/heads/master by this push:
     new c6924da  Define extra before first use
c6924da is described below

commit c6924dafbfbdb9f239be3bcadc9c9785ad6ccd84
Author: Sebb <se...@apache.org>
AuthorDate: Thu Jul 9 00:21:59 2020 +0100

    Define extra before first use
---
 www/roster/views/actions/memstat.json.rb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/www/roster/views/actions/memstat.json.rb b/www/roster/views/actions/memstat.json.rb
index 9d86373..dea8678 100644
--- a/www/roster/views/actions/memstat.json.rb
+++ b/www/roster/views/actions/memstat.json.rb
@@ -21,12 +21,13 @@ if @action == 'emeritus' or @action == 'active' or @action == 'deceased'
       raise Exception.new("Failed to remove existing entry -- try refreshing")
     end
 
+    extra = []
+
     # determine where to put the entry
     if @action == 'emeritus'
       index = text.index(/^\s\*\)\s/, text.index(/^Emeritus/))
       entry.sub! %r{\s*/\* deceased, .+?\*/},'' # drop the deceased comment if necessary
       # if pending emeritus request was found, move it to emeritus
-      extra = []
       # If emeritus request was found, move it to emeritus
       filename = ASF::EmeritusRequestFiles.extractfilename(@emeritusfileurl)
       if filename