python - Model with instance won't save to DB but no error message in Django -

-

i'm having issue saving form instance. have model foreign key want save - nothing appearing in table , there doesn't appear error message.

the model is:

class aboutme(models.model):     mygender                   = models.charfield(max_length=50, choices = gender_choices)   class message(models.model):     mysubject     =  models.charfield(max_length=100)     mymessage     =  models.charfield(max_length=100)     myfromid      =  models.foreignkey(aboutme)  class contactform(modelform):     class meta:         model = message         exclude = ('myread', 'mydeleted', 'myspam', 'mydate', 'mytime', 'myfromid', 'mytoid')     mymessage = forms.charfield(widget=forms.textarea)  def contact(request):         myid = request.session["aboutme_id"]          sender = aboutme.objects.get(pk=myid)         if request.method == "post":             message = contactform(request.post, instance=sender)             if message.is_valid():                 message.save(commit=false)                 #return httpresponse(myid)                 message.myread = false                 message.mydeleted = false                 message.myspam = false                 message.mydate = datetime.date.today()                 message.mytime = timezone.now()                 message.mytoid = int(1)                 message.save()                 message.save_m2m()                 return redirect('/')         else:             message = contactform(request.post, instance=sender)         return render(request, "contact.html", {'form': message,})

i've tried too:

def contact(request):     myid = request.session["aboutme_id"]      sender = aboutme.objects.get(pk=myid)     if request.method == "post":         messageform = contactform(request.post, instance=sender)             if messageform.is_valid():             message = messageform.save(commit=false)             #message.save(commit=false)             #return httpresponse(myid)             message.myread = false             message.mydeleted = false             message.myspam = false             message.mydate = datetime.date.today()             message.mytime = timezone.now()             message.mytoid = int(1)             message.save()             #message.save_m2m()             return redirect('/')     else:         messageform = contactform( instance=sender)     return render(request, "contact.html", {'form': messageform,})

you passing wrong instance form, because form message model, not aboutme model.

here corrected version, along additional checks:

  1. you should make sure have valid session. if have request.session["aboutme_id"] , session has expired, you'll keyerror (as key not in session). using get method of dictionaries suppress error. instead, if key doesn't exists, method return none. if session has expired, should not move forward - redirect root.

  2. even if session valid - may not contain valid value model. in case, aboutme.objects.get(pk=myid) raise exception. prevent that, wrap in try/except block. if object doesn't exists, means value session invalid. delete key , redirect user root.

  3. finally, should have condition if form invalid; added check well.

the rest of changes formatting. method names should start lowercase letters, should normal names. class names should in initialcaps.

def contact(request):     myid = request.session.get("aboutme_id")     if not myid:         # session has expired         return redirect('/')     try:         sender = aboutme.objects.get(pk=myid)     except aboume.doesnotexist:         # invalid id in session         del request.session['aboutme_id']         return redirect('/')      messageform = contactform(request.post or {})     if request.method == "post":               if messageform.is_valid():             message = messageform.save(commit=false)             message.myread = false             message.mydeleted = false             message.myspam = false             message.mydate = datetime.date.today()             message.mytime = timezone.now()             message.mytoid = 1             message.myfromid = sender             message.save()             return redirect('/')         else:             # form not valid             return render(request, "contact.html", {'form': messageform})      else:          # return empty form          return render(request, 'contact.html', {'form': messageform})

Comments

Post a Comment

Popular posts from this blog

c# - How Configure Devart dotConnect for SQLite Code First? -

-
can configure following sample code ? when run following code, error : no entity framework provider found ado.net provider invariant name 'devart.data.sqlite' setting in machine.config <system.data> <dbproviderfactories> <add name="dotconnect sqlite" invariant="devart.data.sqlite" description="devart dotconnect sqlite" type="devart.data.sqlite.sqliteproviderfactory, devart.data.sqlite, version=4.6.287.0, culture=neutral, publickeytoken=09af7300eec23701" /> </dbproviderfactories> </system.data> after add below block code in app.config <?xml version="1.0"?> <configuration>   <configsections> <!-- more information on entity framework configuration, visit http://go.microsoft.com/fwlink/?linkid=237468 --> <section name="entityframework" type="system.data.entity.internal.configfile.entityframeworksection, entit...
Read more

java - Copying object fields -

-
what best way of copying classes when need have 2 independent variables? have simple class: public class myclass { boolean = false; string b= "not empty"; } do need make method : assign(myclass data ) { a= data.a; b= data.b; } is there automatic method copy (duplicate) objects in java? do need make method : pretty close. instead of making method, should make constructor. such constructors called copy constructor , , create them this: myclass(myclass data) { = data.a; b = data.b; } and create copy of instance, use constructor this: myclass obj1 = new myclass(); myclass obj2 = new myclass(obj1); copy constructor can tedious : using copy constructor create deep-copy can tedious, when class has mutable fields. in case, assignment create copy of reference, , not object itself. have create copy of fields (if want deep copy). can go recursive. a better way create deep copy serialize , deserialize object. why not use clone()? ...
Read more

c++ - Clear the memory after returning a vector in a function -

-
i have function processes , stores lot of data in , returns results vector of class. amount of data stored in function tremendous , want clear storage memory of function after finished job. necessary (does function automatically clear memory) or should clear memory function? update: vector<customers> process(char* const *filename, vector<int> id) { vector<customers> list_of_customers; (perform actions) return list_of_customers; } variables defined locally within function automatically released @ end of function scope. destructors objects called, should free memory allocated objects (if objects coded correctly). example of such object std::vector . anything you've allocated new must have corresponding delete release storage. try avoid doing own allocation , use raii instead, i.e. containers or smart pointers.
Read more