Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unexpected THIS_VERSION value #5084

Closed
stevland opened this issue Oct 2, 2019 · 11 comments
Closed

Unexpected THIS_VERSION value #5084

stevland opened this issue Oct 2, 2019 · 11 comments

Comments

@stevland
Copy link

stevland commented Oct 2, 2019

bootstrap.php, Line 340

Expected:
define('THIS_VERSION', '1.12.3'); // Set by installer

Actual:
define('THIS_VERSION', '1.12-git'); // Set by installer


Admin Panel > Settings > System

Expected:
System Settings and Preferences — osTicket (1.12.3)

Actual:
System Settings and Preferences — osTicket (1.12-git)


Intentional?

@JediKev
Copy link
Contributor

JediKev commented Oct 2, 2019

We are looking into it.

@JediKev
Copy link
Contributor

JediKev commented Oct 2, 2019

@stevland

Issue addressed on the packaging side with:

However you will have to wait for the ZIPs to be updated on the website.

Cheers.

@JediKev JediKev closed this as completed Oct 2, 2019
@PeopleInside
Copy link
Contributor

Why the issue is closed if the fix on the website where we can download the packange is still be affected? When the issue will be fixed in the website?

@JediKev
Copy link
Contributor

JediKev commented Oct 2, 2019

@PeopleInside

We are currently re-uploading them.

Cheers.

@PeopleInside
Copy link
Contributor

Hi @JediKev , when the upload is finish i will download it if you alert me so i can fix the issue created from the update. Many thanks. Hope this will not happen again in future update.
Thanks :)

@JediKev
Copy link
Contributor

JediKev commented Oct 2, 2019

@PeopleInside

Hope this will not happen again in future update.

It will not happen again, that's what the pull request mentioned above addresses.

Cheers.

@JediKev
Copy link
Contributor

JediKev commented Oct 2, 2019

@PeopleInside @stevland

Files on website will not be updated until a bit later. In the meantime you can download them here:

Cheers.

@stevland
Copy link
Author

stevland commented Oct 2, 2019

Thanks once again for all of your work and your incredibly fast response times, @JediKev

@joeldeteves
Copy link

Hello @JediKev we are still getting the wrong version. We use a custom Docker image that pulls latest like this:

`git clone -b v${OSTICKET_VERSION} --depth 1 https://github.com/osTicket/osTicket.git \`

Where ${OSTICKET_VERSION} is specified as 1.12.3, etc.

We haven't had this issue with past upgrades.

Per @ntozier's comment in https://forum.osticket.com/d/95656-new-version-1-12-3-install-issue-version-github-from-the-osticket-website/5 it's not feasible for us to download the latest from the website (cause it uses a randomized AWS link).

I tested the command manually and it does appear to pull down 1.12.3 (looking at the WHATSNEW file)

Thank you,

@JediKev
Copy link
Contributor

JediKev commented Nov 5, 2019

@joeldeteves

Are you pulling latest develop or are you using the v1.12.3 version tag? You will need this I believe:

Also, if you download and use straight from github it will show as 1.x-git, always. However, if you deploy the code from the git repo to your html directory you will have the full version. This is best practice anyways so that you can mess with git without affecting production. So, basically you will need to clone from Github to a directory, then run $ php /path/to/clone/manage.php deploy -v /path/to/html/dir/, and then you should have the full version number.

Cheers.

@joeldeteves
Copy link

joeldeteves commented Nov 5, 2019

Thanks Kev, yes we are using the deploy command in our Dockerfile.

I will give that other tag a try.

EDIT: I cloned from the develop branch and checked out the merge request and that solved the issue:

   git clone https://github.com/osTicket/osTicket.git \
   && cd osTicket \
   && git checkout 759cdbdaab3bf0945d50c99d1ae271c15cab9cfd \
   && php manage.php deploy -sv /data/upload

Thank you,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants