Allow all frosh to vote (please test this) (#372)

* Allow all frosh to vote (please test this)

* fixed my stupidity

Author: Mobmaker55

packet/commands.py

@@ -194,7 +194,7 @@ def remove_sig(packet_id: int, username: str, is_member: bool) -> None:
             db.session.commit()
             print('Successfully unsigned packet')
         else:
-            print('Failed to unsign packet; {} is not an onfloor'.format(username))
+            print('Failed to unsign packet; could not find signature')
 
 
 @app.cli.command('remove-member-sig')

packet/routes/shared.py

@@ -27,12 +27,6 @@ def freshman_packet(packet_id, info=None):
     if packet is None:
         return 'Invalid packet or freshman', 404
     else:
-        can_sign = packet.is_open()
-
-        # If the packet is open and the user is an off-floor freshman set can_sign to False
-        if packet.is_open() and app.config['REALM'] != 'csh':
-            if info['uid'] not in map(lambda sig: sig.freshman_username, packet.fresh_signatures):
-                can_sign = False
 
         # The current user's freshman signature on this packet
         fresh_sig = list(filter(
@@ -43,7 +37,6 @@ def freshman_packet(packet_id, info=None):
         return render_template('packet.html',
                                info=info,
                                packet=packet,
-                               can_sign=can_sign,
                                did_sign=packet.did_sign(info['uid'], app.config['REALM'] == 'csh'),
                                required=packet.signatures_required(),
                                received=packet.signatures_received(),

packet/templates/active_packets.html

@@ -20,7 +20,6 @@ <h4 class="page-title">Active Packets</h4>
         </div>
         <div id="eval-blocks">
             {% if packets|length > 0 %}
-                {% set can_sign = info.onfloor or info.realm == "csh" %}
                 <div id="eval-table">
                     <div class="card">
                         <div class="card-body table-fill">
@@ -34,9 +33,7 @@ <h4 class="page-title">Active Packets</h4>
                                         <th>Signatures</th>
                                         <th>Signatures</th>
                                         {% endif %}
-                                        {% if can_sign %}
-                                            <th>Sign</th>
-                                        {% endif %}
+                                        <th>Sign</th>
                                     </tr>
                                     </thead>
                                     <tbody>
@@ -77,7 +74,6 @@ <h4 class="page-title">Active Packets</h4>
                                                 {% endif %}
                                             </td>
                                             {% endif %}
-                                            {% if can_sign %}
                                                 <td class="sign-packet" align="right" data-priority="1">
                                                     {% if not packet.did_sign_result and info.ritdn != packet.freshman_username %}
                                                         <button class="btn btn-sm btn-primary sign-button"
@@ -92,7 +88,6 @@ <h4 class="page-title">Active Packets</h4>
                                                         </button>
                                                     {% endif %}
                                                 </td>
-                                            {% endif %}
                                         </tr>
                                     {% endfor %}
                                     </tbody>

packet/templates/packet.html

@@ -10,7 +10,7 @@
                     <h3>{{ get_rit_name(packet.freshman_username) }}</h3>
                 </div>
                 <div class="col">
-                    {% if can_sign and not did_sign %}
+                    {% if not did_sign %}
                         <button class="btn btn-primary sign-button"
                                 data-packet_id="{{ packet.id }}"
                                 data-freshman_name="{{ get_rit_name(packet.freshman_username) }}">Sign
@@ -105,7 +105,7 @@ <h5>Upperclassmen Score - {{ '%0.2f' % upper_score }}%</h5>
                     </div>
                 </div>
                 {% endif %}
-                {% if info.is_upper or packet.freshman_username == info.ritdn or can_sign %}
+                {% if info.is_upper or packet.freshman_username == info.ritdn %}
                 <div class="card mb-2">
                     <div class="card-header">
                         <b>On-Floor Freshmen Signatures</b>

packet/utils.py

@@ -150,16 +150,9 @@ def sync_freshman(freshmen_list: dict) -> None:
 
     # Update the freshmen signatures of each open or future packet
     for packet in Packet.query.filter(Packet.end > datetime.now()).all():
-        # Handle the freshmen that are no longer onfloor
-        for fresh_sig in filter(lambda fresh_sig: not fresh_sig.freshman.onfloor, packet.fresh_signatures):
-            FreshSignature.query.filter_by(packet_id=fresh_sig.packet_id,
-                                           freshman_username=fresh_sig.freshman_username).delete()
-
-        # Add any new onfloor freshmen
         # pylint: disable=cell-var-from-loop
         current_fresh_sigs = set(map(lambda fresh_sig: fresh_sig.freshman_username, packet.fresh_signatures))
         for list_freshman in filter(lambda list_freshman: list_freshman.rit_username not in current_fresh_sigs and
-                                                          list_freshman.onfloor and
                                                           list_freshman.rit_username != packet.freshman_username,
                                     freshmen_list.values()):
             db.session.add(FreshSignature(packet=packet, freshman=freshmen_in_db[list_freshman.rit_username]))
@@ -207,9 +200,8 @@ def create_new_packets(base_date: date, freshmen_list: dict) -> None:
             sig.drink_admin = member.uid in drink
             db.session.add(sig)
 
-        for onfloor_freshman in Freshman.query.filter_by(onfloor=True).filter(Freshman.rit_username !=
-                                                                              freshman.rit_username).all():
-            db.session.add(FreshSignature(packet=packet, freshman=onfloor_freshman))
+        for frosh in Freshman.query.filter_by(Freshman.rit_username != freshman.rit_username).all():
+            db.session.add(FreshSignature(packet=packet, freshman=frosh))
 
     db.session.commit()