jasig-ssp IRC Logs-2013-07-09

Owned by Apereo Infrastructure
Last updated: Jul 09, 2013
3 min read

[11:30:44 CDT(-0500)] <js70> k

[11:34:08 CDT(-0500)] <JasonElwood> 1473

[11:34:30 CDT(-0500)] <JasonElwood> seems like there is something I'm missing in the description. let me know if you figure it out

[11:34:43 CDT(-0500)] <js70> ok

[14:31:40 CDT(-0500)] <TonyUnicon> hey Jason, quick question about student docs

[14:31:52 CDT(-0500)] <JasonElwood> yep

[14:32:12 CDT(-0500)] <TonyUnicon> 'Name of File' , is that intended to literally be the file name, or is it more like a label?

[14:32:19 CDT(-0500)] <TonyUnicon> https://wiki.jasig.org/display/SSP/SSP+v2.0+Student+Documents

[14:33:19 CDT(-0500)] <TonyUnicon> in other words, to be preserve the name of the original uploaded file, or is this field intended to overwrite that? or more like act like a title or label?

[14:33:28 CDT(-0500)] <TonyUnicon> to we preserve*

[14:33:32 CDT(-0500)] <TonyUnicon> do we preserve*

[14:33:37 CDT(-0500)] <JasonElwood> a label

[14:33:47 CDT(-0500)] <TonyUnicon> ok

[14:33:52 CDT(-0500)] <JasonElwood> please

[14:34:28 CDT(-0500)] <TonyUnicon> also, it's not explicitly listed in the requirements but do we want the ability to delete these?

[14:34:39 CDT(-0500)] <TonyUnicon> nevermind

[14:34:43 CDT(-0500)] <TonyUnicon> it is in one of the mockups

[14:34:47 CDT(-0500)] <JasonElwood> yeah. there's a delete button in the UI

[14:34:48 CDT(-0500)] <TonyUnicon> its not in one I saw

[14:35:39 CDT(-0500)] <TonyUnicon> Dan I have a couple of questions for you

[14:36:22 CDT(-0500)] <TonyUnicon> or actually one more for Jason

[14:36:29 CDT(-0500)] <TonyUnicon> when we 'edit' a student doc

[14:36:50 CDT(-0500)] <TonyUnicon> do we really want to let them change the file

[14:36:59 CDT(-0500)] <TonyUnicon> or can we lock that down and only allow them to change the other fields

[14:37:20 CDT(-0500)] <JasonElwood> other fields only

[14:37:42 CDT(-0500)] <TonyUnicon> sweet, thanks

[14:39:11 CDT(-0500)] <TonyUnicon> Dan, I am using this method to write to disc currently

[14:39:14 CDT(-0500)] <TonyUnicon> http://static.springsource.org/spring/docs/2.0.x/api/org/springframework/web/multipart/commons/CommonsMultipartFile.html#transferTo(java.io.File)

[14:39:29 CDT(-0500)] <TonyUnicon> it does not say if the implementation uses input streams

[14:39:35 CDT(-0500)] <TonyUnicon> do you have any experience in it?

[14:39:39 CDT(-0500)] <TonyUnicon> with it*

[14:40:44 CDT(-0500)] <dmccallum54> i'm looking at the impl. it appears to be safe

[14:40:50 CDT(-0500)] <TonyUnicon> where is it?

[14:40:57 CDT(-0500)] <TonyUnicon> oh you have the src jar?

[14:41:01 CDT(-0500)] <TonyUnicon> i should pull those down

[14:41:05 CDT(-0500)] <dmccallum54> intellij finds it for me (smile)

[14:41:12 CDT(-0500)] <TonyUnicon> bah

[14:41:23 CDT(-0500)] <dmccallum54> there's a mvn opt for doing it...

[14:41:34 CDT(-0500)] <TonyUnicon> yeah there is, ill ask the googles

[14:41:44 CDT(-0500)] <dmccallum54> mvn dependency:sources

[14:42:03 CDT(-0500)] <TonyUnicon> thanks!

[14:46:06 CDT(-0500)] <dmccallum54> the v_external_person view definition on linux ci is stale. doesn't know about the external_person.non_local_address

[14:46:14 CDT(-0500)] <dmccallum54> this sounds familiar… anyone else run into this?

[14:48:08 CDT(-0500)] <TonyUnicon> strange

[14:48:50 CDT(-0500)] <TonyUnicon> looked at the last couple of times we touch that view in liquibase to see if we drop the column

[14:49:04 CDT(-0500)] <TonyUnicon> but we use select * to create the view

[14:52:18 CDT(-0500)] <JasonElwood> wasn't there something when Paul added the student_type

[14:57:29 CDT(-0500)] <TonyUnicon> In terms of what the default list of allowable file types

[14:57:40 CDT(-0500)] <TonyUnicon> did you have a list in mind, or did you want me to cook it up?

[14:59:00 CDT(-0500)] <JasonElwood> xls, xlsx, doc, docx, pdf at least

[14:59:23 CDT(-0500)] <TonyUnicon> gif, jpeg, png ?

[14:59:32 CDT(-0500)] <JasonElwood> yeah

[14:59:57 CDT(-0500)] <JasonElwood> just trying to avoid video, executables, audio (probably)

[15:00:08 CDT(-0500)] <TonyUnicon> I think all we can really hope to do is validate the file extension

[15:00:20 CDT(-0500)] <TonyUnicon> which can easily be circumvented

[15:00:46 CDT(-0500)] <JasonElwood> right, but most users will play by the presented rules

[15:01:05 CDT(-0500)] <TonyUnicon> yep

[16:17:23 CDT(-0500)] <tbain1> Where are the files stored?

[18:32:13 CDT(-0500)] <pspaude1> Those that need Student Types code now have it available.

[18:33:43 CDT(-0500)] <tbain1> ^^

[18:34:33 CDT(-0500)] <pspaude1> It's not 100% you can set it or add new, but its not pre-populating existing values for the code column. Fix coming hopefully shortly.