Just checked again and I'm getting the output I was looking for.
Thanks for the help. I guess the server was just taking a little time
to pick up changes to the URL confs.
On Jan 31, 5:58 pm, Nick <nickt...@gmail.com> wrote:
> Daniel,
>
> Thanks for the quick reply. I have made the adjustments to the URLS
> and the views and am getting a 404 error
>
> Not Found
>
> The requested URL /Government/reps/Prater_David was not found on this
> server. (using the entry for David Prater)
>
> I'm not really sure why it isn't picking up the URL.
>
> On Jan 31, 2:05 pm, Daniel Roseman <dan...@roseman.org.uk> wrote:
>
> > On Jan 31, 7:29 pm, Nick <nickt...@gmail.com> wrote:
>
> > > Thanks in advance for anyone answering this thread.
>
> > > I am building a DB of local government representatives. My goal is to
> > > spit out a general list of the reps and a detailed bio page for each
> > > rep.
>
> > > The URL structure I would like to have is 'government/reps/list' for a
> > > list of all of the Reps and 'government/reps/
> > > REPSLASTNAME_REPSFIRSTNAME' for the single bio view.
>
> > > Here is my model:
> > > class Rep(models.model):
> > > <There is a bunch of dictionaries of the choices that go right here
> > > that I'm ommitting>
> > > Type = models.CharField(max_length=100, choices=Type_Choices,
> > > blank=True)
> > > Last_Name = models.CharField('Last Name', max_length=100,
> > > blank=True)
> > > First_Name = models.CharField('First Name', max_length=100,
> > > blank=True)
> > > Position = models.CharField('Position', help_text="Only used for
> > > non-council City and Local officials", max_length=100, blank=True,
> > > choices=Position_Choices)
> > > Party = models.CharField(max_length=3, choices=Party_Choices,
> > > blank=True)
> > > District = models.IntegerField(help_text="For State, Federal and
> > > County Commissioners", blank=True, null=True)
> > > Ward = models.IntegerField(help_text="For City Councils",
> > > blank=True, null=True)
> > > City = models.CharField(max_length=100, blank=True)
> > > Phone = models.CharField(max_length=15, help_text="Use the form
> > > xxx-xxx-xxxx", blank=True)
> > > Email = models.EmailField(max_length=100, blank=True)
> > > Contact_URL = models.URLField(max_length=200, blank=True)
> > > DOB = models.DateField('Date of Birth',blank=True, null=True)
> > > Gender = models.CharField(blank=True, max_length=2,
> > > choices=Gender_Choices)
> > > Begin_Serve = models.IntegerField('Began Serving in', blank=True,
> > > null=True)
> > > End_Serve = models.IntegerField('Term Limited in', blank=True,
> > > null=True)
> > > MugShot = models.ImageField('Mug Shot Upload', help_text="images
> > > are to be no larger that 150x200", storage=s3_storage,
> > > upload_to='newsok/images/Government/images/mugs', height_field=None,
> > > width_field=None, max_length=300, blank=True)
> > > Committees = models.ManyToManyField('Committees', blank=True)
> > > Approp_Committee = models.ManyToManyField('Approp_Committees',
> > > blank=True)
>
> > > def __unicode__(self):
> > > return u"%s, %s" % (self.Last_Name, self.First_Name)
>
> > > class Meta:
> > > abstract = False
>
> > > Here is my view:
>
> > > from Government.Reps.models import *
> > > from django.shortcuts import render_to_response
> > > from django.http import Http404
>
> > > def index(request):
> > > rep_list_view = Rep.objects.all().order_by('Last_Name')
> > > return render_to_response('Government/repsAll.html', {'allReps':
> > > rep_list_view})
>
> > > def detail(request, (Last_Name, First_Name)):
> > > try:
> > > r = Rep.objects.all().order_by('Last_Name', 'First_Name')
> > > except Rep.DoesNotExist:
> > > raise Http404
> > > return render_to_response('Government/repsSingle.html',
> > > {'singleRep': r})
>
> > > Here is my URL conf:
>
> > > from django.conf.urls.defaults import *
>
> > > # Uncomment the next two lines to enable the admin:
> > > from django.contrib import admin
> > > admin.autodiscover()
>
> > > urlpatterns = patterns('',
> > > # Example:
> > > # (r'^Government/', include('Government.foo.urls')),
>
> > > # Uncomment the admin/doc line below and add
> > > 'django.contrib.admindocs'
> > > # to INSTALLED_APPS to enable admin documentation:
> > > # (r'^admin/doc/', include('django.contrib.admindocs.urls')),
>
> > > # Uncomment the next line to enable the admin:
> > > (r'^admin/', include(admin.site.urls)),
> > > (r'^reps/(P?<Last_Name>_<First_Name>\d+)/$',
> > > 'Government.Reps.views.detail'),
>
> > > I think that I am either overthinking this or way UNDER thinking it.
>
> > > This is my first go at a template being called via variables passed in
> > > the URL. I have a good background in templates and the models portion
> > > of django but am sorely lacking in my understanding of views and URL
> > > confs.
>
> > > Thanks again,
> > > Nick
>
> > You're on the right track. A couple of things though.
>
> > Firstly, your urlconf. You need to learn a bit about regular
> > expressions: \d+ matches one or more numeric digits, which obviously
> > isn't what you want. And you have the P? the wrong way round - it
> > should be ?P. Also, you want first name and last name to be sent as
> > separate variables, so you want one (?P ) group per variable. So what
> > you need is this:
> > (r'^reps/(?P<last_name>\w+)_(?P<first_name>\w+)/$',
> > 'Government.Reps.views.detail'),
> > Although note that this won't match names like O'Connell, as \w
> > doesn't match an apostrophe. Perhaps this is better:
> > (r'^reps/(?P<last_name>[A-Za-z']+)_(?P<first_name>[A-Za-z]+)/$',
> > 'Government.Reps.views.detail'),
>
> > Secondly, in your view, you have some extra brackets in the function
> > definition:
> > def detail(request, last_name, first_name):
>
> > And you want to use .get() to get the actual matching Rep:
> > r = Rep.objects.get(Last_Name=last_name, First_Name=first_name)
>
> > Note that this will raise an exception if a matching Rep is not found.
> > You may want to make this show a 404 page if that happens, in which
> > case you can use the handy get_object_or_404 shortcut:
>
> > from django.shortcuts import get_object_or_404
> > r = get_object_or_404(Rep, Last_Name=last_name,
> > First_Name=first_name)
>
> > Finally, although this isn't really an error, you should take note of
> > PEP8 to use the Pythonic style of capitalisation. Class attributes (eg
> > field names) and normal variables are usually spelled all lower case,
> > which is why I've used last_name and first_name above.
> > --
> > DR.
--
You received this message because you are subscribed to the Google Groups
"Django users" group.
To post to this group, send email to django-us...@googlegroups.com.
To unsubscribe from this group, send email to
django-users+unsubscr...@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/django-users?hl=en.
