OpenEMR 3.0.1

tmccormi wrote on Monday, April 06, 2009:

Now that it’s officially released I recommend that we put anything we find that is a ‘bug’ in the Tracker with “3.0.1” as the first thing on the subject  line (unless one of the admins wants to add Release 3.0.1 to the Group selection).

That way we can separate Bugs that need to be fixed, qa’ed and released as patches from Enhancements and development experiments.

What do you think?  It’s a decent tool, better that the forum for tracking this kind of stuff.

–Tony

drbowen wrote on Tuesday, April 07, 2009:

OMG!

The OpenEMR 3.0.1 Windows release is awesome!

Thanks Brady, Joe, Richard, and Obinna for the setup scripts that made this possible.

Sam Bowen, MD

tmccormi wrote on Tuesday, April 07, 2009:

Version id 3.0.0.1 versus 3.0.1 …

The product release says it 4 digits, the threads say 3.

The release manager in me askes why 4 digits

3 = Primary Major Enhancement Release
0 = Minor Enhancement Release
1 = Bug Patch release

The postfix ‘dev’ seems to be used for unstable developer releases

So what’s the designated purpose of the 4th (or the redundant 3rd)?

–Tony

bradymiller wrote on Tuesday, April 07, 2009:

Tony,

Added Release 3.0.1 to groups in tracker.

You are correct.  The version on the production release 3.0.1 should be 3.0.1 (it’s a bug, should not be 3.0.0.1 which is what we use for development version).

We’ve been following it seems below:
3 = Supreme release
0 = Major release
1 = Minor release
1 = Development release

-brady

drbowen wrote on Wednesday, April 08, 2009:

I fixed this for the Widows packages:

xampp-openemr-3.0.1.zip
openemr-3.0.1.zip

Both come up saying 3.0.1.

Looking at globals.php.  there appears to be a typo?

[code]
//Version tags

$v_major = ‘3’;
$v_minor = ‘0’;
$v_patch = ‘1’;
$tag = ‘.1’; // minor revision number, should be empty for production releases

// This name appears on the login page and in the title bar of most windows.
// It’s nice to customize this to be the name of your clinic.
$openemr_name = ‘OpenEMR’;

$openemr_version = "$v_major.$v_minor.$v_patch".$tag;   // Version tag used by program (line 179)
[/code]

Shouldn’t the double quote “.$tag by after the .$tag”  ?

I don’t know that much but it seems wrong to me.

I got this to work in windows by commenting out line 173:

//$tag = ‘.1’; // minor revision number, should be empty for production releases

Sam Bowen

bradymiller wrote on Wednesday, April 08, 2009:

Sam,

   The coding is ugly, but it’s not a bug.  The bug was we just forgot to change the version before packaging it up as a release. We usually just make $tag variable empty, no need to comment out.

    It’s ok to change it in your xampp version since user is aware this is not original release with some modifications. 

    But, we definitely should not change anything in the openemr-3.0.1.zip release, since this is being downloaded with the tar.gz file as the official 3.0.1 release.  I’d suggest making a openemr-3.0.1.zip release with no mods to place in sourceforge (download the tar.gz file, extract it, then make the zip); then everybody will be downloading the “official” release.  I hope this makes sense.

-brady

ideaman911 wrote on Wednesday, April 08, 2009:

Guys;

Just a thought from a moron - if there are two files which are supposedly the same, inevitably one will get overlooked, at which point they won’t be.

Given that 7zip is free and can untar to a zip, would we not be FAR better to put a link and instruction to download 7zip for Windows users to make it easier on the programming community?

I know I am a great proponent of making things as simple as possible for the end user.  But I am also a quality guy.  Just a thought.

Joe Holzer    Idea Man
http://www.holzerent.com

ajonate wrote on Wednesday, April 08, 2009:

Another thought.

If the only difference between the tarball and the “Windows version” is the zip file format, then it’s not really a Windows version. As long as we have it why not really make the Windows version a Windows version by providing a proper path in the globals.php file? It wouldn’t take that much effort to do it, and it’s not like a Linux path will do a Windows user any good.

bradymiller wrote on Wednesday, April 08, 2009:

Ajonate,
  The path in globals.php is now basically auto-configured for user during installation script.
-brady

drbowen wrote on Wednesday, April 08, 2009:

There is no modification with the following exceptions:

xampp-openemr
Installed with xampp
I fixed the subversion number to 3.0.1
I changed the Concurrent_Layout to "2"

openemr 3.0.1
I fixed the subversion number to 3.0.1
I changed the Concurrent_Layout to "2"

I think the "official" release should be changed to either Concurrent_Layout version "0" or "2".

Since we have the tree view it doesn’t make sense to stick with the much uglier radio buttons.

I for me and mine always use the original layout.  It is much easier to train and instruct new users.  The tree view is very nice for experienced users.

Sam Bowen, MD

drbowen wrote on Wednesday, April 08, 2009:

If these changes are not acceptable as "official"  I would argue that we need to make these changes and release 3.0.2 as a second "minor" bug fix.

Sam Bowen, MD

ideaman911 wrote on Wednesday, April 08, 2009:

Guys;

My testing suggests that the view will change with a globals.php change in the code (0-2) and a logout-login.  Any reason we couldn’t put that preference for view on the login screen?  That would allow “preferences” to work for all users, rather than limited to the preference of the “wizard of release” :wink:

Joe Holzer    Idea Man
http://www.holzerent.com