My favorites | Sign in
Project Home Source
READ-ONLY: This project has been archived. For more information see this post.
Search
for
  Advanced search   Search tips   Subscriptions
Issue 250: trying to delete students, tells me i need benchmark related tables
2 people starred this issue and may be notified of changes. Back to list
 
Project Member Reported by ca...@burkesoftware.com, Dec 2, 2011
Deleting or saving students throws an error that it requires "benchmark_grade_mark", "benchmark_grade_aggregate", and "benchmark_grade_aggregate_student".

For schools that do not have the benchmark gradebook, this shouldn't happen. They shouldn't need these tables.
Dec 2, 2011
Project Member #1 jmil...@cristoreyny.org
I made "Bogus, Kid" in the NY instance and see the deletion error.

I can change data and save, though. Maybe you were changing a different
attribute than I was?

On Fri, Dec 2, 2011 at 2:19 PM, <
student-worker-relational-database@googlecode.com> wrote:
Dec 2, 2011
Project Member #2 ca...@burkesoftware.com
deleting faculty errors with "benchmark_grade_category", "benchmark_grade_item", and "benchmark_grade_aggregate_course" tables missing.
Dec 2, 2011
Project Member #3 ca...@burkesoftware.com
maybe it was just delete then. I've been doing so much the last couple of
days, I might've screwed up

On Fri, Dec 2, 2011 at 2:33 PM, <
student-worker-relational-database@googlecode.com> wrote:
Dec 3, 2011
Project Member #4 dmbs...@gmail.com
The issue is that benchmark_grade/models.py is being read in to python due to some import statements outside the benchmark_grade project without checks if it's installed. 

For example sis/report.py line 9

The proper way to access classes from another django app that isn't guaranteed to be present is to check installed apps then import just what you need. Don't use a global import or an import *. Something like

if 'foo' in settings.INSTALLED_APPS:
   from foo.models import JustThisModel
   JustThisModel(stuff)
   ...
Dec 4, 2011
Project Member #5 jmil...@cristoreyny.org
This issue was closed by revision 43cabc1f9fd8.
Status: Fixed
Dec 4, 2011
Project Member #6 jmil...@cristoreyny.org
Thanks for the report and the help.
Dec 13, 2011
Project Member #7 jmil...@cristoreyny.org
 Issue 258  has been merged into this issue.
Cc: da...@burkesoftware.com
Dec 13, 2011
Project Member #8 jmil...@cristoreyny.org
Broken by revision c3516bdeaf
Status: Started
Dec 13, 2011
Project Member #9 dmbs...@gmail.com
merge fail?
Dec 13, 2011
Project Member #10 jmil...@cristoreyny.org
This issue was closed by revision 260ab38ec426.
Status: Fixed

Powered by Google Project Hosting