Opened 13 years ago

Closed 12 years ago

#159 closed defect (fixed)

After verifying email; the link to view user gallery doesn't work

Reported by: Odin Hørthe Omdal Owned by:
Priority: minor Milestone:
Component: programming Keywords:
Cc: Parent Tickets:

Description

It's actually bare. Why? Because ``verify_email`` that calls
``templates/mediagoblin/user_pages/user.html`` doesn't set
``user_gallery_url``.

Instead of toying with the ``verify_email`` view, I rather made the
user model be able to make its own URL, using the urlgen method
other stuff use. It's pretty verbose, but I guess that's because we
don't want to import urlgen in the model?

Not beautiful, but now the user.html template will work in from any
view that gives it the user object.

Obviously, one could also just add ``user_gallery_url`` to
``verify_email``, but that doesn't really seem right. Shouldn't
need to toss around such easy derived information.



Attachments (1)

verify_email-fix_user_gallery_url.patch (2.2 KB ) - added by Odin Hørthe Omdal 13 years ago.
verify_email-fix_user_gallery_url.patch

Download all attachments as: .zip

Change History (4)

by Odin Hørthe Omdal, 13 years ago

verify_email-fix_user_gallery_url.patch

comment:1 by Odin Hørthe Omdal, 12 years ago

Status: NewResolved
WFM now. Couldn't find a broken link in the latest version...



comment:2 by Will Kahn-Greene, 12 years ago

The original url for this bug was http://bugs.foocorp.net/issues/449 .

comment:3 by Jakob Kramer, 12 years ago

Component: programming
Resolution: fixed
Status: acceptedclosed
Note: See TracTickets for help on using tickets.