Help save net neutrality! Learn more.
Close

#339 4.1.0 new install gives sql error on login

Release_4.1.0
open
nobody
5
2012-07-17
2012-07-17
Rod Roark
No

After doing a plain old install of 4.1.0 (from the rel-410 branch at SF) and logging in, I get this:

ERROR: query failed: SELECT count(dr.dr_id) c FROM `dated_reminders` dr JOIN `users` u ON dr.dr_from_ID = u.id JOIN `dated_reminders_link` drl ON dr.dr_id = drl.dr_id WHERE drl.to_id = ? AND dr.`message_processed` = 0 AND dr.`dr_message_due_date` < ADDDATE(NOW(), INTERVAL 5 DAY)
Error: Table 'openemr410.dated_reminders' doesn't exist

Seems the missing SQL setup is in sql/patch.sql you're supposed to do an upgrade from admin.php to run it. That should not be necessary. The SQL should be in database.sql.

Discussion

  • Brady Miller

    Brady Miller - 2012-07-17

    Hi Rod,

    This has been discussed on the forums. This sort of just happened as the patching mechanism was built. Agreed it makes sense to modify the database.sql also. Main issues are that it doesn't really make sense to do for OpenEMR 4.1.0 since nearing end of this release and I'm also a bit hesitant to do for 4.1.1 simply because it requires more testing (ie. resources) on my part to test (need to ensure the patch works and the raw install works).

    -brady

     
  • Rod Roark

    Rod Roark - 2012-07-17

    Hi Brady, I recall a discussion about database updates for the patch process, but what I'm saying is that if you grab the current rel-410 code and do a normal install it doesn't work. I don't think that's a problem in 4.1.1 because the dated reminder table there is set up by database.sql.

     
  • Brady Miller

    Brady Miller - 2012-07-18

    Hi,
    The rel-410 isn't set up to install like that (the patch script needs to be run after running setup (note that even the 4.1.0 demo with most updated patch does this). I've just been avoiding it because having to test both the patch and a straight install would be to time consuming for myself; to ensure robust basically tell users to install the official 4.1.0 packages and then run the patch script. For 4.1.1 I am likely to try out (ie. see how much addiitonal timne it takes) making the rel-411 branch work on a native install, so may potentially be able to just do installs for 4.1.1 patches.
    -brady

     

Log in to post a comment.