search

zak-hassan / WhoTeachsWhat

JBoss JSP Application for Data Management
0 stars 0 forks source link

Remove Course Crossover and old_name from CourseController, CourseService #136

Closed zak-hassan closed 10 years ago

zak-hassan commented 10 years ago

Since we are not managing which course go in a program then why do we need to have CourseController and CourseService?

In the database create script we don't have properties for these fields....

ghost commented 10 years ago

We need this because course codes are changing so Mary-Lynn wants to be able to know what course was what and as for crossover, a professor who is assigned, say GAM666 is also assigned DPS918. They are a very similar course.

Also we absolutely need a CourseController and CourseService. How else are we going to add course, update and remove courses? We definitely need it. The only thing debatable was Program and everything related to that. Everything else is a must.

zak-hassan commented 10 years ago

We have a CourseController but I need to update the sql script to regenerate the correct models. Would you like to fix this or should I ?

ghost commented 10 years ago

The models have private String crossover_course private String old_course

We shouldn't need to do anything but update the latest script with those fields. I can do that.

This issue doesn't seem to have a problem anymore. I will close it.

zak-hassan commented 10 years ago

Done