subscribers: modify functions for 'user' argument

* Necessary for bulk subscription feature where current_user undefined.

hepdata/modules/records/subscribers/api.py

@@ -31,12 +31,14 @@
 from .models import Subscribers
 
 
-def is_current_user_subscribed_to_record(recid):
-    if not current_user.is_authenticated:
-        return False
+def is_current_user_subscribed_to_record(recid, user=None):
+    if not user:
+        user = current_user
+        if not current_user.is_authenticated:
+            return False
 
     return Subscribers.query.filter(Subscribers.publication_recid == recid,
-                                    Subscribers.subscribers.contains(current_user)).count() > 0
+                                    Subscribers.subscribers.contains(user)).count() > 0
 
 
 def get_users_subscribed_to_record(recid):

hepdata/modules/records/subscribers/rest.py

@@ -34,13 +34,17 @@ def list_subscriptions_for_user():
 
 
 @blueprint.route('/subscribe/<int:recid>', methods=['POST'])
-@login_required
-def subscribe(recid):
+def subscribe(recid, user=None):
+    if not user:
+        user = current_user
+        if not current_user.is_authenticated:
+            return jsonify({"success": False, "status_code": 500})
+
     record_subscribers = get_or_create(db.session, Subscribers, publication_recid=recid)
 
     try:
-        if not current_user in record_subscribers.subscribers:
-            record_subscribers.subscribers.append(current_user)
+        if not user in record_subscribers.subscribers:
+            record_subscribers.subscribers.append(user)
 
         db.session.add(record_subscribers)
         db.session.commit()