Merge pull request #580 from Gig-o-Matic/lockable_plans

Lockable plans

docker-compose.yml

@@ -1,5 +1,4 @@
 ---
-version: "3.8"
 services:
   web:
     build: .

gig/helpers.py

@@ -60,6 +60,8 @@ def decorated(request, pk, *args, **kw):
 @login_required
 @plan_editor_required
 def update_plan(request, plan, val):
+    if plan.gig.plans_locked:
+        return HttpResponseForbidden()
     plan.status = val
     plan.save()
     return render(request, 'gig/plan_icon.html', {'plan_value': val})
@@ -127,7 +129,7 @@ def generate_changes(latest, previous):
     #     changes.append((_('Set Time'), *date_diff(latest.setdate, previous.setdate)))
     # if 'enddate' in diff.changed_fields:
     #     changes.append((_('End Time'), *date_diff(latest.enddate, previous.enddate)))
-        
+
     check = [x in diff.changed_fields for x in ['is_full_day', 'date', 'setdate', 'enddate', 'datenotes']]
     if True in check:
         changes.append((_('Date/Time'), _('(See below.)'), None))
@@ -143,7 +145,7 @@ def generate_changes(latest, previous):
 
     if 'setlist' in diff.changed_fields:
         changes.append((_('Set List'), None, None))
-    
+
     if 'dress' in diff.changed_fields:
         changes.append((_('What To Wear'), _('(See below.)'), None))
 
@@ -203,6 +205,20 @@ def notify_new_gig(gig, created, dates=None):
         async_task('gig.helpers.send_email_from_gig', gig,
                 'email/new_gig.md' if created else 'email/edited_gig.md')
 
+@login_required
+@band_editor_required
+def gig_lock_plans(request, gig):
+    gig.plans_locked = True
+    gig.save()
+    return redirect('gig-detail', pk=gig.id)
+
+@login_required
+@band_editor_required
+def gig_unlock_plans(request, gig):
+    gig.plans_locked = False
+    gig.save()
+    return redirect('gig-detail', pk=gig.id)
+
 @login_required
 @band_editor_required
 def gig_untrash(request, gig):
@@ -246,7 +262,7 @@ def create_gig_series(the_gig, number_to_copy, period):
         delta = timedelta(weeks=1)
     else:
         day_of_month = last_date.day
-        
+
     set_delta = (the_gig.setdate - the_gig.date) if the_gig.setdate else None
     end_delta = (the_gig.enddate - the_gig.date) if the_gig.enddate else None
 
@@ -265,7 +281,7 @@ def create_gig_series(the_gig, number_to_copy, period):
             last_date = last_date.replace(month=mo, day=min(calendar.monthrange(yr,mo)[1], day_of_month), year=yr)
 
         the_gig.date = last_date
-        
+
         if set_delta is not None:
             the_gig.setdate = the_gig.date + set_delta
 
@@ -278,6 +294,6 @@ def create_gig_series(the_gig, number_to_copy, period):
         the_gig.cal_feed_id = uuid.uuid4()
         the_gig.save()
         the_dates.append(the_gig.date)
-    
+
     # return the list of dates for all the gigs
     return the_dates

gig/migrations/0021_gig_plans_locked_historicalgig_plans_locked.py

@@ -0,0 +1,23 @@
+# Generated by Django 4.2.17 on 2025-02-16 22:13
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('gig', '0020_auto_20240325_1946'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='gig',
+            name='plans_locked',
+            field=models.BooleanField(default=False),
+        ),
+        migrations.AddField(
+            model_name='historicalgig',
+            name='plans_locked',
+            field=models.BooleanField(default=False),
+        ),
+    ]

gig/migrations/0025_merge_20250222_1612.py

@@ -0,0 +1,14 @@
+# Generated by Django 4.2.17 on 2025-02-22 16:12
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('gig', '0021_gig_plans_locked_historicalgig_plans_locked'),
+        ('gig', '0024_change_dates_by_band_timezone'),
+    ]
+
+    operations = [
+    ]

gig/models.py

@@ -52,10 +52,10 @@ def future_plans(self, member):
 
         # find plans that are for this member that are not trashed or archived
         possible = super().get_queryset().filter(assoc__member=member,
-                                             assoc__status=AssocStatusChoices.CONFIRMED,
-                                             gig__trashed_date__isnull=True,
-                                             gig__is_archived=False,
-                                            )
+                                                 assoc__status=AssocStatusChoices.CONFIRMED,
+                                                 gig__trashed_date__isnull=True,
+                                                 gig__is_archived=False,
+                                                )
         # find plans for gigs that haven't ended yet
         possible = possible.filter((Q(gig__is_full_day=True) & Q(gig__date__gte=yesterday_for_user)) |
                                    (Q(gig__enddate=None) & Q(gig__date__gte=recent_for_user)) |
@@ -192,12 +192,8 @@ class Gig(AbstractEvent):
     leader = models.ForeignKey('member.Member', blank=True, null=True, related_name="leader_gigs", on_delete=models.SET_NULL)
     leader_text = models.TextField(null=True, blank=True)
 
-    # todo manage these
-    # trueenddate = ndb.ComputedProperty(lambda self: self.enddate if self.enddate else self.date)
-    # sorttime = ndb.IntegerProperty( default=None )
-
-    # todo what's this?
-    # comment_id = ndb.TextProperty( default = None)
+    # Flag whether band members can change their plans
+    plans_locked = models.BooleanField(default=False)
 
     rss_description = models.TextField(null=True, blank=True)
 

gig/urls.py

@@ -30,6 +30,8 @@
     path('<int:pk>/untrash', helpers.gig_untrash, name='gig-untrash'),
     path('<int:pk>/archive', helpers.gig_archive, name='gig-archive'),
     path('<int:pk>/remind', helpers.gig_remind, name='gig-remind'),
+    path('<int:pk>/lock_plans', helpers.gig_lock_plans, name='gig-lock-plans'),
+    path('<int:pk>/unlock_plans', helpers.gig_unlock_plans, name='gig-unlock-plans'),
     path('<int:pk>/printallplans', views.PrintPlansView.as_view(), {'all':True}, name='gig-print-all-plans'),
     path('<int:pk>/printconfirmedplans', views.PrintPlansView.as_view(), {'all':False}, name='gig-print-confirmed-plans'),
     path('<int:pk>/printsetlist', views.PrintSetlistView.as_view(), name='gig-print-setlist'),