Merge branch 'bugfix/bugfixes' into develop

rowers/dataprep.py

@@ -1123,7 +1123,6 @@ def save_workout_database(f2, r, dosmooth=True, workouttype='rower',
                 inboard=inboard, oarlength=oarlength,
                 timezone=timezone_str,
                 privacy=privacy)
-
     try:
         w.save()
     except ValidationError:
@@ -1394,6 +1393,7 @@ def new_workout_from_file(r, f2,
         
     id, message = save_workout_database(
         f2, r,
+        notes=notes,
         workouttype=workouttype,
         weightcategory=r.weightcategory,
         adaptiveclass=r.adaptiveclass,

rowers/templates/document_form.html

@@ -39,7 +39,9 @@
           {{ form.as_table }}
         </table>
         {% csrf_token %}
-        <input class="button green" type="submit" value="Submit">
+        <p>
+          &nbsp;<input type="submit" value="Submit">
+        </p>
     </div>
 
   </li>
@@ -201,10 +203,16 @@ $('#id_workouttype').change();
       if ($(this).attr('id') == 'id_file') {
         data.set("file",this.files[0]);
     }
-    console.log($(this).attr('name'),$(this).val());
+    console.log($(this).attr('id'),$(this).attr('name'),$(this).val());
   };
       });});
 
+  $('textarea').each(function( i ) {
+  $(this).change(function() {
+    data.set($(this).attr('name'),$(this).val());
+    console.log($(this).attr('id'),$(this).attr('name'),$(this).val());
+  });});      
+  
   $('select').each(function( i ) {
   console.log($(this).attr('name'),$(this).val());
   $(this).change(function() {

rowers/urls.py

@@ -111,7 +111,6 @@ def response_error_handler(request, exception=None):
     return HttpResponse('Error handler content', status=403)
 
 def filenotfound_handler(request, exception=None):
-    print('noot')
     return HttpResponse('Error handler content', status=404)
 
 handler403 = views.error403_view
@@ -381,11 +380,11 @@ urlpatterns = [
     re_path(r'^workout/(?P<id>\b[0-9A-Fa-f]+\b)/addstatic/(?P<plotnr>\d+)/$',
         views.workout_add_chart_view,name='workout_add_chart_view'),
     re_path(r'^workout/(?P<id>\b[0-9A-Fa-f]+\b)/addstatic/$',views.workout_add_chart_view,name='workout_add_chart_view'),
-    re_path(r'^workout/(?P<pk>\d+)/delete/$',login_required(
+    re_path(r'^workout/(?P<pk>\b[0-9A-Fa-f]+\b)/delete/$',login_required(
         views.WorkoutDelete.as_view()),
         name='workout_delete'),
-    re_path(r'^workout/(?P<id>\b[0-9A-Fa-f]+\b)/delete/$',login_required(
+#    re_path(r'^workout/(?P<id>\b[0-9A-Fa-f]+\b)/delete/$',login_required(
-        views.workout_code_delete_view),name='workout_code_delete'),
+#        views.workout_code_delete_view),name='workout_code_delete'),
     re_path(r'^workout/(?P<id>\b[0-9A-Fa-f]+\b)/smoothenpace/$',views.workout_smoothenpace_view,name='workout_smoothenpace_view'),
     re_path(r'^workout/(?P<id>\b[0-9A-Fa-f]+\b)/undosmoothenpace/$',views.workout_undo_smoothenpace_view,name='workout_undo_smoothenpace_view'),
     re_path(r'^workout/c2import/$',views.workout_c2import_view,name='workout_c2import_view'),

rowers/views/workoutviews.py

@@ -3900,6 +3900,11 @@ def workout_upload_view(request,
     except KeyError:
         boattype = '1x'
 
+    try:
+        notes = docformoptions['notes']
+    except KeyError:
+        notes = ''
+
     try:
         workoutsource = uploadoptions['workoutsource']
     except KeyError:
@@ -4028,7 +4033,7 @@ def workout_upload_view(request,
                     boattype=boattype,
                     makeprivate=makeprivate,
                     title = t,
-                    notes=''
+                    notes=notes,
                 )
             else:
                 workoutsbox = Mailbox.objects.filter(name='workouts')[0]
@@ -5289,7 +5294,11 @@ class GraphDelete(DeleteView):
 def workout_code_delete_view(request,id=0):
     pk = encoder.decode_hex(id)
 
+    try:
+        w = Workout.objects.get(pk=pk)
         url = reverse('workout_delete',kwargs={'pk':pk})
+    except Workout.DoesNotExist:
+        url = reverse('workout_delete',kwargs={'pk':id})
 
     return HttpResponseRedirect(url)
     
@@ -5341,7 +5350,16 @@ class WorkoutDelete(DeleteView):
         return reverse('workouts_view')
 
     def get_object(self, *args, **kwargs):
-        obj = super(WorkoutDelete, self).get_object(*args, **kwargs)
+        workout_pk = self.kwargs['pk']
+        try:
+            obj = Workout.objects.get(pk=workout_pk)
+        except (ValueError,Workout.DoesNotExist):
+            workout_pk = encoder.decode_hex(workout_pk)
+            try:
+                obj = Workout.objects.get(pk=workout_pk)
+            except Workout.DoesNotExist:
+                raise Http404("One of the workouts doesn't exist")
+                #        obj = super(WorkoutDelete, self).get_object(*args, **kwargs)
         if not checkaccessuser(self.request.user,obj.user):
             raise PermissionDenied('You are not allowed to delete this workout')