Implemented editing for model instances
This commit is contained in:
@@ -4,4 +4,4 @@ from .models import Player
|
||||
class PlayerForm(forms.ModelForm):
|
||||
class Meta:
|
||||
model = Player
|
||||
fields = ['first_name', 'last_name', 'jersey_number']
|
||||
fields = ['first_name', 'last_name', 'jersey_number', 'team']
|
||||
@@ -5,5 +5,6 @@ from . import views
|
||||
urlpatterns = [
|
||||
path('', views.root, name="root"),
|
||||
path('list', views.list, name="players list"),
|
||||
path('edit/<int:id>', views.edit, name="edit player")
|
||||
path('edit/<int:id>', views.edit, name="edit player"),
|
||||
path('edit', views.edit, name="edit player")
|
||||
]
|
||||
@@ -13,17 +13,22 @@ def list(request):
|
||||
'items': [(player.id, f"{player.first_name} {player.last_name}") for player in players],
|
||||
'edit_url_name': 'edit player'})
|
||||
|
||||
def edit(request, id=None):
|
||||
def edit(request, id=0):
|
||||
# if this is a POST request we need to process the form data
|
||||
if request.method == 'POST':
|
||||
# create a form instance and populate it with data from the request:
|
||||
form = PlayerForm(request.POST)
|
||||
if id:
|
||||
instance = get_object_or_404(Player, id=id)
|
||||
form = PlayerForm(request.POST or None, instance=instance)
|
||||
else:
|
||||
form = PlayerForm(request.POST or None)
|
||||
# check whether it's valid:
|
||||
if form.is_valid():
|
||||
# process the data in form.cleaned_data as required
|
||||
# ...
|
||||
# redirect to a new URL:
|
||||
return HttpResponse(str(form.cleaned_data))
|
||||
new_player, did_create = Player.objects.update_or_create(pk=id, defaults=form.cleaned_data)
|
||||
return render(request, 'success.html', {'call_back':'players list'})
|
||||
|
||||
# if a GET (or any other method) we'll create a blank form
|
||||
else:
|
||||
@@ -33,4 +38,4 @@ def edit(request, id=None):
|
||||
else:
|
||||
form = PlayerForm
|
||||
|
||||
return render(request, 'players/edit.html', {'form': form, 'id': id})
|
||||
return render(request, 'edit.html', {'form': form, 'id': id, 'call_back':'edit player'})
|
||||
Reference in New Issue
Block a user