Guarding against zero length string

[jpodivin]

No description provided.

[TomasTomecek]

Unfortunely this haven't fixed the issue for me. Locally in chrome (tried private browsing) it still behaves the same :/

I naively tried chrome's dev tools but couldn't see the content of the current-hash-atom and current-hash variables? functions?

[TomasTomecek]

This one works really well! Thank you for fixing this.

Let's link this PR with #149

[FrostyX]

Thank you very much for the fix @jpodivin.
For the record, the bug is in:

(def current-hash-atom (r/atom (or (current-hash) "#copr")))

because empty string isn't considered a false value and therefore (or "" "foo") doesn't return "foo" but the empty string. I didn't know that. So the "#copr" default is never used. The proposed fix works, so +1 for that, but it can be simpler:

--- a/frontend/src/app/homepage.cljs
+++ b/frontend/src/app/homepage.cljs
@@ -18,7 +18,9 @@
 (def input-values (r/atom nil))
 (def input-errors (r/atom []))
 (def backend-stats (r/atom nil))
-(def current-hash-atom (r/atom (or (current-hash) "#copr")))
+(def current-hash-atom
+  (r/atom (if (empty? (current-hash)) "#copr" (current-hash))))
+
 (def report-target (r/atom 1000))
 (def error (r/atom nil))