Comments (8)
How do you pass None
to a form from the browser? Or are you using a form programmatically?
from django-recurrence.
I found it during testing yes so the form data was being generated and passed programmatically, I basically added recurrence: ''
to the POST.data
to fix it, but I expected if I didn't pass the field name to the form post data it wouldn't crash. I wasn't doing recurrence: None
in the post data, I didn't include it at all. (The RecurrenceField
is called recurrence
on my model) So the form cleaning that was done assigns a value of None
to the field if it is not present.
from django-recurrence.
Ah, OK - that makes sense, thanks @audiolion.
Pull requests to fix this welcome - especially if they contain tests!
from django-recurrence.
Ok might be a few days but I will send in a patch
from django-recurrence.
Sounds good - thanks! No pressure on getting it done, it doesn't sound like it would hit most users of django-recurrence. Don't worry if you can't get tests for it - I'll happily add them to a patch if needed.
from django-recurrence.
Did this ever get patched? I'm running into this issue now. Thanks.
Edit: I monkey-patched it for now, so no rush. I returned None if a TypeError was thrown in the try/catch, seems to work okay for now, as my Recurrence field is not required.
from django-recurrence.
@harveynr - no, I don't think this was ever patched.
from django-recurrence.
Fixed by #122.
from django-recurrence.
Related Issues (20)
- Is there a "include_dtend" parameter? HOT 2
- Translation is not working for other languages.
- Read the Docs configuration update HOT 1
- Implement Jazzband guidelines for django-recurrence HOT 1
- Wagtail Error HOT 2
- Recurrence Models
- The between() method with inc=True includes dtstart even when there is no Recurrence then HOT 1
- How to install this app? HOT 1
- Support for DEFAULT_AUTO_FIELD HOT 2
- No error handling in JS code if translation is not setup correctly HOT 1
- Subset of supported rules not specified.
- Monthly Last Days (-1, -2, -3, -4) Highlight issue
- function to_dateutil_rruleset() include the dtend in returned array HOT 2
- Migration bug "Nonexistent parent node"
- Saving in 'UTC' problem with DST HOT 1
- Remove use of deprecated ‘distutils’ library HOT 1
- Multiple Recurrence widgets on the same page cause a JS mess
- Abandoned? HOT 1
- Version 1.12 isn't listed on PyPI
- Retrigger pipeline to publish 1.12
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from django-recurrence.