Opened 15 years ago

Closed 14 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 15 years ago.
verify_email-fix_user_gallery_url.patch

Download all attachments as: .zip

Change History (4)

by Odin Hørthe Omdal, 15 years ago

verify_email-fix_user_gallery_url.patch

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

Status: NewResolved

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

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

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

comment:3 by Jakob Kramer, 14 years ago

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