Skip to end of metadata
Go to start of metadata

You are viewing an old version of this page. View the current version.

Compare with Current View Page History

« Previous Version 30 Next »

[09:49:59 CST(-0600)] <rickard> How do you override user editable "portlet.xml Preferences"? There's no "Edit configuration" link or anything like that in portlet manager.
[09:53:19 CST(-0600)] <holdorph> in portlet manager there should be an option to go to a portlet preferences screen
[09:53:27 CST(-0600)] <holdorph> on that screen click the 'edit preferences' button
[09:53:38 CST(-0600)] <holdorph> then 'add parameter'
[09:54:27 CST(-0600)] <holdorph> fill in the value, then 'review', then 'save'
[09:58:23 CST(-0600)] <rickard> Hmm
[09:59:18 CST(-0600)] <rickard> Can't seem to find the edit preferences button.
[10:00:57 CST(-0600)] <holdorph> http://dl.dropbox.com/u/998658/prefs.png
[10:01:02 CST(-0600)] <holdorph> is what it looks like for me
[10:01:04 CST(-0600)] <rickard> http://www.ch.lu.se/portlet-manager.png
[10:01:37 CST(-0600)] <EricDalquist> ah so if the portlet has config mode, which that one does, you can't edit the preferences directly
[10:01:38 CST(-0600)] <rickard> Is it because this portlet supports "rich configuration"?
[10:01:45 CST(-0600)] <holdorph> yup
[10:01:47 CST(-0600)] <rickard> Ok
[10:01:58 CST(-0600)] <holdorph> yeah, i'm not a big fan of that, but it is what it is
[10:01:59 CST(-0600)] <rickard> So then I have to modify the portlet.xml and redeploy?
[10:02:12 CST(-0600)] <EricDalquist> that is on purpose to prevent breaking the portlet config generated by the config mode
[10:02:24 CST(-0600)] <holdorph> yeah, but you're in the admin screens
[10:02:36 CST(-0600)] <holdorph> it's not like you can't break the portlet a ton of other ways
[10:02:45 CST(-0600)] <EricDalquist> true
[10:02:55 CST(-0600)] <EricDalquist> perhaps add it back in with a big warning?
[10:03:03 CST(-0600)] <holdorph> it just makes my job harder if i know what I really want to do
[10:03:18 CST(-0600)] <holdorph> it's ESPECIALLY annoying when config mode decides to break for a point release or two
[10:03:28 CST(-0600)] <rickard> (big grin)
[10:03:59 CST(-0600)] <holdorph> big warning would be fine. i'm sure admins are used to seeing those.
[10:09:01 CST(-0600)] <rickard> athena: Hate to bug you even more about this, but just so we're clear. Even though adding the charset stuff to rdbm.properties solved my main utf8 issues with the portlet, it still doesn't work with cleanContent=true. Just type a non-ascii character in the editor and hit preview to see what I mean.
[10:09:15 CST(-0600)] <athena> ok
[10:09:25 CST(-0600)] <athena> i suspect that may be an issue with tomcat's default encoding
[10:09:31 CST(-0600)] <athena> and yes, i've seen what you'er talkign about
[10:09:54 CST(-0600)] <rickard> Hmm, ok.
[10:10:55 CST(-0600)] <rickard> Do you mean the URIEncoding stuff we talked about the other day?
[10:14:43 CST(-0600)] <rickard> I'll look into it.
[10:16:20 CST(-0600)] <rickard> Another quick question. What's the best way to completely uninstall and remove a portlet? I can remove the folder from the webapps folder in Tomcat and delete the portlet using portlet manager, men what about from the uPortal source? If I redeploy uPortal it'll put them back in, right?
[10:17:19 CST(-0600)] <holdorph> if it's one of the ones shipped with uportal, yup. you'd have to edit the (looks up the file name)
[10:18:15 CST(-0600)] <holdorph> uportal-portlets-overlay/pom.xml and remove the entry from the modules list
[10:18:28 CST(-0600)] <holdorph> i think that's the simplest way, although I haven't tested that theory (wink)
[10:18:36 CST(-0600)] <rickard> (big grin)
[10:18:41 CST(-0600)] <rickard> I'll give it a go.
[10:18:43 CST(-0600)] <rickard> Thanks.
[10:19:10 CST(-0600)] <holdorph> that's only in addition to the two things you mentioned (removing the web app and deleting from portlet manager)
[10:19:29 CST(-0600)] <rickard> Ok
[10:21:14 CST(-0600)] <rickard> athena: Adding URIEncoding="UTF-8" to the Connector in server.xml did indeed solve the antisamy stuff. Weird that the URI encoding would affect this. I mean, is the portlet passing the data to AntiSamy via GET?
[10:23:06 CST(-0600)] <athena> yep
[10:23:13 CST(-0600)] <athena> and via post
[10:23:18 CST(-0600)] <athena> which i'm a bit more concerned about
[10:23:29 CST(-0600)] <athena> the content gets checked twice
[10:23:37 CST(-0600)] <athena> first when you exist "preview" mode before you save
[10:23:51 CST(-0600)] <rickard> Ah, ok.
[10:23:55 CST(-0600)] <athena> that ajax operation is designed to give you an accurate preview of your content - so you know right away if antisamy is going to delete some key element
[10:24:05 CST(-0600)] <rickard> Makes sense.
[10:24:07 CST(-0600)] <athena> then of course the whole form gets posted so the back-end code can save it
[10:24:31 CST(-0600)] <athena> and we re-parse it there to make sure it hasn't changed, though i'm not actually that worried about a privileged administrator hacking the page to get around the preview
[10:24:37 CST(-0600)] <athena> good practice to double check though
[10:26:06 CST(-0600)] <rickard> Yeah, but if it turns out to be a problem for i18n, it might be worth skipping.
[10:26:10 CST(-0600)] <rickard> In my opinion.
[10:26:37 CST(-0600)] <rickard> If a malicious user has admin access, we're screwed anyway.
[10:29:42 CST(-0600)] <rickard> I take that back. Adding URIEncoding="UTF-8" did not solve it. It makes it work with characters from latin1 (e.g Γ₯, Γ€, ΓΆ etc), but not multibyte characters.
[10:29:48 CST(-0600)] <rickard> Man, this is confusing.
[10:32:28 CST(-0600)] <rickard> Oh well, time to head home. Have a good weekend.
[10:33:02 CST(-0600)] <athena> i suspect that what we really need is UTF-8 support for POST parameters
[10:33:49 CST(-0600)] <rickard> I believe we have that.
[10:34:17 CST(-0600)] <rickard> As long as the browser considers the page on which the form resides to be UTF8, it should post the data as UTF8.
[10:34:44 CST(-0600)] <rickard> That is unless tomcat does something strange behind the scenes.
[10:37:29 CST(-0600)] <rickard> Bye
[10:37:35 CST(-0600)] <athena> actually a tomcat problem
[10:37:41 CST(-0600)] <athena> but we can talk next week - have a great weekend!
[10:37:45 CST(-0600)] <athena> thanks again for all your help on this
[13:21:14 CST(-0600)] <EricDalquist> drooling a little over the spring security 3.0 annotations: http://static.springsource.org/spring-security/site/docs/3.0.x/reference/el-access.html#el-pre-post-annotations
[13:47:26 CST(-0600)] <athena> yeah, combined with EL that stuff gets super cool
[13:47:44 CST(-0600)] <athena> would love to move to spring security
[14:57:12 CST(-0600)] <awills> Happy Friday
[14:57:22 CST(-0600)] * athena cheers heartily
[14:57:37 CST(-0600)] <awills> just posetd this (note also sent to list): https://issues.jasig.org/browse/UP-2927
[14:58:16 CST(-0600)] * holdorph wonders if awills knows about the new #jasig-uportal channel
[14:58:18 CST(-0600)] <awills> I think this is a wanted feature... seeking input to best align this work with the preferences and plans of others in the community
[14:58:34 CST(-0600)] <EricDalquist> using either channel is fine right now (smile)
[14:58:44 CST(-0600)] <EricDalquist> they both get logged to the same location
[14:58:49 CST(-0600)] <awills> news to me... amking a note
[14:59:09 CST(-0600)] <EricDalquist> I'll setup the forward to move people into #jasig-uportal this weekend
[14:59:11 CST(-0600)] <athena> yes, i think that's totally a wanted feature
[14:59:22 CST(-0600)] <EricDalquist> yes I agree
[14:59:30 CST(-0600)] <athena> should add that channel to my auto-joins
[14:59:33 CST(-0600)] <awills> step thrrough the patch, if you have a few minutes
[14:59:45 CST(-0600)] <EricDalquist> will do
[14:59:52 CST(-0600)] <awills> awesomeness
[15:00:18 CST(-0600)] <athena> woo features!
[15:00:23 CST(-0600)] <athena> think that really makes sense
[15:00:32 CST(-0600)] <athena> and hopefully shouldn't be a big lift
[15:02:06 CST(-0600)] <EricDalquist> looks good
[15:02:57 CST(-0600)] <awills> very cool... i'll have to do a bit of rework for the 3.3. version... but that's fine
[15:03:15 CST(-0600)] <EricDalquist> it would be neat to just integrate it into the gallery ui in trunk
[15:03:38 CST(-0600)] <EricDalquist> maybe talk with matt/gary and see if they have ideas on where a good place to put a reset link would be
[15:03:53 CST(-0600)] <awills> do you mean a link to it? or the feature itself?

  • No labels