[12:08:19 CDT(-0500)] <JasonElwood> JS- are there any other conditions for the MAP plan status except a row in external_person_planning_status?
[12:40:03 CDT(-0500)] <js70> not at this time
[14:29:43 CDT(-0500)] <JasonElwood> JS- so I've tested this back and forth. I'm having problems with both the plan status and dev course indicator. A rows is returned with this query for the student I'm using. select * from external_person_planning_status where school_id = 'jgarcia35'
[14:31:18 CDT(-0500)] <JasonElwood> And the is_dev indicator doesn't appear to be saved on the plan. This query returns the course with is_dev = . select * from external_course where is_dev = 'Y' Is the value in external_course supposed to be some other value? It does show as false in map_plan_course when saved to a plan.
[14:32:48 CDT(-0500)] <js70> I'll check but I'm pretty sure is_dev = "Y" is correct.
[14:33:16 CDT(-0500)] <js70> hold on let me look.
[14:34:28 CDT(-0500)] <js70> is_dev is boolean on the PlanCourse
[14:34:43 CDT(-0500)] <js70> there may be a translaction issue.
[14:36:10 CDT(-0500)] <js70> I just tested that the indicator worked if planCourse is_dev was true
[14:37:30 CDT(-0500)] <JasonElwood> it's definitely is_dev = Y in external_course. when that course is dragged to a plan, is_dev = f in map_plan_course
[14:38:39 CDT(-0500)] <js70> is_dev = ? in map_plan_course?
[14:38:51 CDT(-0500)] <JasonElwood> false
[14:39:17 CDT(-0500)] <JasonElwood> when I manually update it to true, the indicator shows up. looks like the value isn't being carried over from external_course
[14:39:31 CDT(-0500)] <js70> yep, thats whats happening
[14:39:49 CDT(-0500)] <js70> didn't think to test that part.
[14:39:52 CDT(-0500)] <js70> like a dope.
[14:40:04 CDT(-0500)] <JasonElwood> but I know that has worked because I've seen the Dev Cr Hrs total show more than zero
[14:40:33 CDT(-0500)] <js70> yeah, if the plan_course_dev was set by hand perhaps?
[14:41:21 CDT(-0500)] <JasonElwood> if is_dev in external_course should be Y or N, then we have a problem. And, I'll open a ticket
[14:43:52 CDT(-0500)] <js70> we have a problem external bools are standardized on Y,N operational appears to be boolean.
[14:45:04 CDT(-0500)] <dmccallum54> we have hql querying that external field directly with true/false?
[14:46:39 CDT(-0500)] <JasonElwood> the calculation of Dev Hrs on the plan used to work
[14:47:29 CDT(-0500)] <js70> No, pretty sure this is all client side concerns.
[14:47:43 CDT(-0500)] <JasonElwood> I created 1454
[14:48:48 CDT(-0500)] <JasonElwood> the were another question about the planning_status. similar situation where the row exists in the database, but the status on the MAP shows "Student is Currently: No Status"
[14:49:13 CDT(-0500)] <JasonElwood> I can create a ticket if you don't think it is a data issue
[14:50:19 CDT(-0500)] <JasonElwood> external_person_planning_status.status = OFF or ON
[14:50:39 CDT(-0500)] <js70> better create a ticket
[14:50:47 CDT(-0500)] <JasonElwood> np
[14:54:23 CDT(-0500)] <JasonElwood> re-opened 968
[14:54:34 CDT(-0500)] <js70> k
[15:09:36 CDT(-0500)] <JasonElwood> JS- is there any scenario where a course in a start term of the Move Plan feature would not move?
[15:09:58 CDT(-0500)] <js70> yeah, when its in the past
[15:10:09 CDT(-0500)] <js70> or its a transcript
[15:10:42 CDT(-0500)] <js70> everything in spring 2013 and earlier should not move
[15:11:03 CDT(-0500)] <js70> any transcripted course thats in the CORRECT term will not move.
[15:11:21 CDT(-0500)] <JasonElwood> even if the course is a planned course?
[15:11:31 CDT(-0500)] <js70> but right now, even if its not in the transcript, if its in the past it won't be moved.
[15:11:37 CDT(-0500)] <js70> yep
[15:11:39 CDT(-0500)] <JasonElwood> past I'm good with
[15:11:45 CDT(-0500)] <JasonElwood> that should never move
[15:11:57 CDT(-0500)] <js70> kool. your idea :^)
[15:12:23 CDT(-0500)] <dmccallum54> i know that's what the spec says, but does that make sense for templates?
[15:12:41 CDT(-0500)] <js70> good question, wondered that myself
[15:12:49 CDT(-0500)] <dmccallum54> "oops, i forgot to bump the template i made last semester before the semester elapsed!"
[15:13:20 CDT(-0500)] <JasonElwood> I can't remember what I documented, but we discussed allowing anything for templates
[15:14:03 CDT(-0500)] <JasonElwood> I'm thinking about the is_transcripted not moving. that means a course is on the plan AND in the transcript right???
[15:14:11 CDT(-0500)] <js70> could be, detailed reading of documentation is not a forte of mine
[15:14:26 CDT(-0500)] <js70> no
[15:14:29 CDT(-0500)] <JasonElwood> I'll check it
[15:14:53 CDT(-0500)] <js70> wait a sec,
[15:15:00 CDT(-0500)] <dmccallum54> i'm fairly certain at the last discussion we agreed no moving of planned stuff in the past for plans or templates
[15:15:05 CDT(-0500)] <js70> yes that is correct but only the plan course will be on the page
[15:15:15 CDT(-0500)] <js70> or in the plan.
[15:15:26 CDT(-0500)] <js70> the course just knows it has a transcripted course.
[15:16:07 CDT(-0500)] <JasonElwood> it makes sense. I think my sample data is just throwing it off.
[15:16:49 CDT(-0500)] <JasonElwood> for moving past terms in templates, the documentation is only for current and future terms
[15:16:51 CDT(-0500)] <js70> there is a possible hickup since transcripted courses use formattedCourse while the plan_course use the code.
[15:18:01 CDT(-0500)] <js70> plan_course use formattedCourse to match with formattedCourse in transcript but its my understanding code guarantees uniqueness.
[15:18:22 CDT(-0500)] <JasonElwood> I think it works fine, just didn't expect it
[15:18:40 CDT(-0500)] <js70> k
[15:19:20 CDT(-0500)] <JasonElwood> I'll test this, but for moving templates … aren't we allowing for manual moving of courses in past terms?
[15:20:05 CDT(-0500)] <js70> short answer - yes
[15:20:55 CDT(-0500)] <JasonElwood> as a user, I would expect the move plan to move courses in past terms because I can manually move them. Make sense to everybody else?
[15:23:36 CDT(-0500)] <js70> you mean courses that are not in transcript?
[15:23:56 CDT(-0500)] <JasonElwood> no, for templates. allow the start term to be a past term.
[15:24:06 CDT(-0500)] <js70> sure, that makes sense.
[15:24:19 CDT(-0500)] <JasonElwood> good catch guys
[16:53:40 CDT(-0500)] <pspaude1> Anyone else having issues with i000038.xml not running due to failure in precondition? I'm on postgres and my db is clear of previous references.
[16:58:02 CDT(-0500)] <dmccallum54> what is the actual error
[17:19:44 CDT(-0500)] <pspaude1> I can't get back to the log statement again. I'm stuck now without it loading at all, it's saying db_username_liquibase can't be found.
[17:20:17 CDT(-0500)] <pspaude1> Anyways, before it said it wasn't executing due to the sql check in the precondition of something of that nature.
[17:23:51 CDT(-0500)] <dmccallum54> i see the problem for db_username_liquibase. one sec
[17:37:20 CDT(-0500)] <dmccallum54> pspaude1 try again?
[17:39:39 CDT(-0500)] <pspaude1> Alright, trying it now.
[17:50:30 CDT(-0500)] <pspaude1> Ok looks like your fix worked. No db_username issues.
[17:50:39 CDT(-0500)] <dmccallum54> preconditions still failing?
[17:51:58 CDT(-0500)] <pspaude1> Yes, its skipping it based on the precondition. No errors. It just isn't doing a good job of checking if the table exists. Because the table and v_table doesn't exist nor is the xml in my changelog.
[17:53:18 CDT(-0500)] <dmccallum54> i000038.xml still?
[17:54:03 CDT(-0500)] <pspaude1> Yes, am looking at it now.
[17:54:14 CDT(-0500)] <dmccallum54> k. in middle of build. can check in a bit
[17:56:33 CDT(-0500)] <pspaude1> Hmm it's not the check I thought it was, its the standard manage integration database precondition.
[17:57:04 CDT(-0500)] <tbain1> I just did a simple write-up of the solution I have for SSP-1411 as a comment on the parent ticket; https://issues.jasig.org/browse/SSP-1402 - let me know if you think there is anything I need to clarify - and I apologize if it's a little weird, I typed it up during a Dev forum so it was difficult to concentrate.
[17:59:28 CDT(-0500)] <dmccallum54> tbain1 thx
[17:59:37 CDT(-0500)] <dmccallum54> can you re-write it in scala now, pls