Admin does not correctly prefill DataTimeField from URL

[gilhad]

I was not able to format URL for Admin interface to prefill DateTimeField with given value.

It worked in 0.96, but does not work in 1.0 ( I used ../admin/MyApp/MyTable/add/?box=359&datum_date=2008-12-01&datum_time=17:30:27)

After some looking on source code and testing i find a solution:

• in /django/contrib/admin/options.py before line 520 add

    if isinstance(f, models.DateTimeField):
           initial[k] = initial[k].split(",")
  

• use this format: ​https://../admin/MyApp/MyTable/add/?box=359&datum=2008-12-01,17:30:27

For reference - the model of MyTable is such :

class MyTable(models.Model):
        box = models.ForeignKey(Boxes)
        datum =  models.DateTimeField(null=True, blank=True)

(plus some other insignificant fields.
The "datum" field should be prefilled with some date, which is computed by long way (not simple now()) and the use must be able to edit it BEFORE saving it)

---

The problem arises from DateTimeField be treated by MultiWidget, but not properly broken when got by URL (GET)

---

Patch:

--- options.py.old      2008-12-01 19:56:34.000000000 +0100
+++ options.py  2008-12-01 19:40:34.000000000 +0100
@@ -517,6 +517,8 @@
                     continue
                 if isinstance(f, models.ManyToManyField):
                     initial[k] = initial[k].split(",")
+                if isinstance(f, models.DateTimeField):
+                    initial[k] = initial[k].split(",")
             form = ModelForm(initial=initial)
             for FormSet in self.get_formsets(request):
                 formset = FormSet(instance=self.model())

[gilhad]

Typo - (Admin does NOT correctly prefill DataTimeField from URL (GET) + patch)

[Malcolm Tredinnick]

Edited description for clarity.

Please attach a proper patch to the ticket, rather than mixing the proposed changes into the description of the problem. It makes things a lot easier for reviewing and further improvements (if subsequent patches are made).

[gilhad]

Added patch for code AND documentation

[Alexander Herrmann]

attached a new diff for the current source and added test

[Gandalfar]

patch against r17364

[Gandalfar]

I've tested that patch works correctly with current trunk and cleaned-up docs syntax.

[Aymeric Augustin]

I dislike the idea of blessing hand-crafted URLs as a public API.

[Tim Graham]

In any case, the patch needs to be updated to apply cleanly.

[Ridley Larsen]

Replying to timo:

In any case, the patch needs to be updated to apply cleanly.

I have attached a patch that applies cleanly and passes testing.

[Claude Paroz]

Ridley, would it be possible to turn the patch into a Github pull request?

[Tim Graham]

​PR

[Markus Holtermann]

It feels to me like using a single white space instead of a comma is more intuitive. 2015-04-05 12:34:56 vs 2015-04-05,12:34:56

[Ridley Larsen]

patch against a2e56db

[Ridley Larsen]

I've attached a new patch with a different approach. Instead of special-casing the SplitDateTimeWidget, I modified its' decompress method to account for string values. Thanks Tim for the advice on the isinstance call.

[Tim Graham]

#19431 is a duplicate with an alternate approach that might be worth looking at to see if any bits could be incorporated.

[Gilhad]

I updated to Djago 1.10 on more sites, the problem still remains (cannot prefill admins datetime field) and the original fix still works (just the lines moved after line 1404 now).

Would be possible to somehow resolve it and incorporate it in main stream, so I would not be forced to manually patch it each and every version? Please, prettty, pretty please ...