[Clipart] Some updated functionality of clipart detail links and bug report combo

Brad Phillips brad at bradphillips.org
Mon Aug 15 01:59:27 PDT 2011


Hey chovynz,

Good work.  I've added the inherit to your requested comment widgets & updated their sql to remove the join dependency.  Also, I updated the management bar links to use the new widget.

I'll let you close out that bug, if this resolves the rest of your issues with that.

Cheers,
+Brad

On Aug 14, 2011, at 7:16 PM, chovynz wrote:

> Hi all
> 
> I will also be sending this to LP, but I wanted to let Jon and Brad know specifically.
> 
> Bug Description:
> Links to clipart detail pages are not consistent throughout the site. The many different links inside the code are tedious and annoying to update.
> 
> Partial Bug Fix:
> I have gone throughout the site and updated many clipart detail links to a new widget, so that if the structure of links to clipart detail pages needs to gets changed, you only need to do it once (inside the new widget)! A good example of this is the recent trouble we had with pretty url's and no comments displaying. This update will allow very quick changes to the entire site, wherever there is a link to clipart detail pages.
> 
> New Widget:
> 1885 link-to-detail
> 
> Content of new widget:
> < a href="[root]/detail/((id))/((link))" >
> 
> Affected widgets:
>  (replaced each hardcoded link with (#(inherent:1885|SELECT * FROM ocal_files WHERE id = ((id)) LIMIT 1)#))
> 31 clipart-thumb
> 163 bydate
> 23 fortags-img (sysGod version)
> 145 fortags-img  X 2 (content and authorised content)
> 103 user-detail
> 
> TODO:
> (my) old SQL is blocking the completion of this bugfix, and I don't know the "ocalfiles.users" way. I would appreciate it if someone could update the sql and content to be compatible with this new widget/method in widgets:
> 1117 latest-comments-box
> 1147 list-personal-comments
> Management Action bar Ajax is something I don't know where to update the clipart detail link. I have had a look around and cannot find where to change the link to this new method. Brad please make the link point to this widget the same way as in bydate.  -> (#(inherent:1885|SELECT * FROM ocal_files WHERE id = ((id)) LIMIT 1)#)
> OCAL became "unavaliable" while I was working on it. Hopefully the changes have remained. I wasn't doing anything that would make the site crash so I don't think it was me. I will add more once OCAL becomes alive again.
>  
> -- 
> Cheers
> Chovynz
> _______________________________________________
> clipart mailing list
> clipart at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/clipart

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/clipart/attachments/20110815/0674e2cf/attachment.html>


More information about the clipart mailing list