Help installing new release of icd9 file

system · October 20, 2013, 6:42pm

bradymiller wrote on Sunday, October 20, 2013:

Hi,

Just committed Stephen’s contribution to upgrade to newer icd9/icd10 codesets to the official codebase:

Thank you Stephen for the contribution (and thanks to Mac, the original developer of the loader code, for making it so straightforward)

-brady
OpenEMR

system · December 2, 2013, 4:01am

bradymiller wrote on Monday, December 02, 2013:

Hi,

Just committed another contribution from Stephen to upgrade to the newer 2013 icd10 codesets to the official codebase:

Again, thank you Stephen for the contribution (and again, thanks to Mac, the original developer of the loader code, for making it so straightforward)

-brady
OpenEMR

system · December 10, 2013, 4:28pm

yehster wrote on Tuesday, December 10, 2013:

If both the cmsv30_master_descriptions.zip and cmsv31-master-descriptions.zip are both present in the contrib/icd9 directory

The error message isn’t particularly helpful to the average user.

The staged files release dates are not all from the same release.

Deleting cmsv30_master_descriptions.zip allows import to happen, but a 4.1.2-4.1.3 patch isn’t going to be able to take care of that.

system · December 11, 2013, 1:54am

cmswest wrote on Wednesday, December 11, 2013:

system · December 11, 2013, 2:53am

yehster wrote on Wednesday, December 11, 2013:

I’m ok with using the most reliable and simplest implementation. It would just be nice if the error message were improved.

Something like:
The staged files release dates are not all from the same release. Please review the files in “…”

system · December 11, 2013, 3:14am

cmswest wrote on Wednesday, December 11, 2013:

ok, “error message isn’t particularly helpful” makes sense for sure

so you’d like something more informative here:

github.com

openemr/openemr/blob/master/interface/code_systems/list_staged.php#L267


      <div class="error_msg"><?php echo xlt("The number of staged files is incorrect. Only place the file that you wish to install/upgrade to."); ?></div>
      <div class="stg msg"><?php echo xlt("Follow these instructions for installing or upgrading the following database") . ": " . text($db); ?><span class="msg" id="<?php echo attr($db); ?>_instrmsg">?</span></div>
      <?php
      $success_flag=0;
  }


// Ensure all release dates and revisions are the same for multiple file imports
// and collect the date and revision. Also collect a checksum and path.
  $file_revision_date = '';
  $file_revision = '';
  $file_checksum = '';
  $file_revision_path = '';
  foreach ($revisions as $value) {
      // date check
      $temp_file_revision_date = $value['date'];
      if (empty($file_revision_date)) {
          $file_revision_date = $temp_file_revision_date;
      } else {
          if (($file_revision_date != $temp_file_revision_date) && ($success_flag === 1)) {
              ?>
              <div class="error_msg"><?php echo xlt("The staged files release dates are not all from the same release."); ?></div>

system · December 11, 2013, 3:15am

cmswest wrote on Wednesday, December 11, 2013:

I remember mentioning this to Brady but he said it had to be that way for a reason.

system · December 11, 2013, 3:16am

cmswest wrote on Wednesday, December 11, 2013:

oh right, you’d like to see an improvement here:

github.com

openemr/openemr/blob/master/interface/code_systems/list_staged.php#L267


      <div class="error_msg"><?php echo xlt("The number of staged files is incorrect. Only place the file that you wish to install/upgrade to."); ?></div>
      <div class="stg msg"><?php echo xlt("Follow these instructions for installing or upgrading the following database") . ": " . text($db); ?><span class="msg" id="<?php echo attr($db); ?>_instrmsg">?</span></div>
      <?php
      $success_flag=0;
  }


// Ensure all release dates and revisions are the same for multiple file imports
// and collect the date and revision. Also collect a checksum and path.
  $file_revision_date = '';
  $file_revision = '';
  $file_checksum = '';
  $file_revision_path = '';
  foreach ($revisions as $value) {
      // date check
      $temp_file_revision_date = $value['date'];
      if (empty($file_revision_date)) {
          $file_revision_date = $temp_file_revision_date;
      } else {
          if (($file_revision_date != $temp_file_revision_date) && ($success_flag === 1)) {
              ?>
              <div class="error_msg"><?php echo xlt("The staged files release dates are not all from the same release."); ?></div>

system · December 11, 2013, 3:17am

cmswest wrote on Wednesday, December 11, 2013:

system · December 11, 2013, 6:46am

bradymiller wrote on Wednesday, December 11, 2013:

It will get taken care of during a 4.1.2->4.1.3 upgrade, though, since in an upgrade the contrib/icd9 is replaced with the newer 4.1.3 codebase.
-brady

system · September 23, 2014, 6:33am

bradymiller wrote on Tuesday, September 23, 2014:

Hi,

Just committed a contribution from Stephen (he has done this for the last 3 years) to upgrade to the newer 2014 icd9 and icd10 codesets to the official codebase:

Again, thank you Stephen for the contribution (and again, thanks to Mac, the original developer of the loader code, for making it so straightforward)

-brady
OpenEMR