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

Theme: Cloudmin version detected incorrectly. #452

Closed
rubenszolt opened this issue Apr 30, 2016 · 16 comments
Closed

Theme: Cloudmin version detected incorrectly. #452

rubenszolt opened this issue Apr 30, 2016 · 16 comments
Labels

Comments

@rubenszolt
Copy link

On fresh install cloudmin GPL with virtualmin GPL, detect my cloudmin version incorectly PRO.
Cloudmin version 8.4.kvm Pro

@iliajie
Copy link
Collaborator

iliajie commented Apr 30, 2016

Ok, I'll take a look.

Thanks for reporting.

@iliajie iliajie added the bug label Apr 30, 2016
@iliajie iliajie changed the title Not detect the corect clodmin version Theme: Cloudmin version detected incorrectly Apr 30, 2016
@iliajie
Copy link
Collaborator

iliajie commented Apr 30, 2016

You can change it in theme settings already.

@iliajie
Copy link
Collaborator

iliajie commented Apr 30, 2016

Once again, you have installed GPL versions only?

@iliajie
Copy link
Collaborator

iliajie commented Apr 30, 2016

Do one more thing please. Open page source and find data-virtual-server-license and data-server-manager-license, what are the values for it?

@iliajie
Copy link
Collaborator

iliajie commented Apr 30, 2016

..but you must open the source of the left frame by clicking on the navigation menu and selecting the show source..

@rubenszolt
Copy link
Author

rubenszolt commented Apr 30, 2016

Yes I installed just GPL version.
data-virtual-server-license="0"
data-server-manager-license="1"

@rubenszolt
Copy link
Author

Today I updated the cloudmin-server-manager to 9.0.kvm GPL version, and same issue.

@iliajie
Copy link
Collaborator

iliajie commented May 2, 2016

Ok, I'll take a look.

@iliajie
Copy link
Collaborator

iliajie commented May 26, 2016

Please test it with 18.00. I suppose it's still not displaying it correctly? If not just report here and I will do a fix look in 18.01.

@rubenszolt
Copy link
Author

rubenszolt commented May 26, 2016

Same issue, i tested under 18.00 version
http://imgur.com/yROOsZs

@iliajie
Copy link
Collaborator

iliajie commented Jun 2, 2016

If you switch back to default (old) theme, what do you see??

What is the output of /usr/share/webmin/server-manager/module.info or for RHEL systems /usr/libexec/webmin/server-manager/module.info?

@iliajie iliajie closed this as completed Jun 2, 2016
@iliajie
Copy link
Collaborator

iliajie commented Jun 2, 2016

Will be fixed in 18.01. But please double check and tell what is the output of above.

@rubenszolt
Copy link
Author

Hi, today i updated to 18.02 version and same issue (BUG), I install authentic theme to other two servers and not detect the corect Cloudmin licence.

/usr/share/webmin/server-manager/module.info output
desc=Cloudmin GPL for KVM
category=webmin
depends=servers net proc 1.480
os_support=*-linux
version=9.0.kvm
onehost=kvm
vm2=kvm
gpl=1
desc_cz=Cloudmin GPL pro KVM
desc_cz.UTF-8=Cloudmin GPL pro KVM

@rubenszolt rubenszolt changed the title Theme: Cloudmin version detected incorrectly Theme: Cloudmin version detected incorrectly. Jun 11, 2016
@iliajie
Copy link
Collaborator

iliajie commented Jun 11, 2016

Alright, I will test it later and come back to this issue.

@iliajie iliajie reopened this Jun 11, 2016
@iliajie
Copy link
Collaborator

iliajie commented Jun 11, 2016

Can you find in authentic-lib.pl

sub licenses {
    my ($id) = @_;
    if ( &foreign_available("virtual-server") && $id eq "vm" ) {
        my %virtualmin = &get_module_info("virtual-server");
        if ( $virtualmin{'version'} =~ /gpl/i ) {
            return 0;
        }
        else {
            return 1;
        }
    }
    elsif ( &foreign_available("server-manager") && $id eq "cm" ) {
        my %cloudmin = &get_module_info("server-manager");
        if ( $cloudmin{'version'} =~ /gpl/i ) {
            return 0;
        }
        else {
            return 1;
        }
    }
    else {
        return 0;
    }
}

And replace it with:

sub licenses {
    my ($id) = @_;
    if ( &foreign_available("virtual-server") && $id eq "vm" ) {
        my %virtualmin = &get_module_info("virtual-server");
        if ( $virtualmin{'version'} =~ /pro/i ) {
            return 1;
        }
        else {
            return 0;
        }
    }
    elsif ( &foreign_available("server-manager") && $id eq "cm" ) {
        my %cloudmin = &get_module_info("server-manager");
        if ( $cloudmin{'version'} =~ /pro/i ) {
            return 1;
        }
        else {
            return 0;
        }
    }
    else {
        return 0;
    }
}

That part seems to be making the problem. Reboot Webmin afterwards. Does it fix the issue?

@iliajie
Copy link
Collaborator

iliajie commented Jun 11, 2016

I have updated 18.02 which will fix your bug. Just force upgrade.

@iliajie iliajie closed this as completed Jun 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants