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

[Suggestion] Add Armory #67

Open
Malachiel87 opened this issue Nov 25, 2017 · 3 comments
Open

[Suggestion] Add Armory #67

Malachiel87 opened this issue Nov 25, 2017 · 3 comments

Comments

@Malachiel87
Copy link

Hi i would like to have a working wow armory maybe also compatible with wow 1.12.2

@paintballrefjosh
Copy link
Owner

Definitely a good idea. I'll add it as an enhancement and to the to-do list.

@Malachiel87
Copy link
Author

Malachiel87 commented Dec 3, 2017

The armory link of old mangos is here https://storage.googleapis.com/google-code-archive-downloads/v2/code.google.com/mwenhanced/MwEnhanced_Armory_3.3.2.rar

And the sql+istructions are uploaded here armory.zip

After converted all script from mysql_ to mysqli_ (php 7) i keep to get the error: "Unable to connect to armory DB of realm blabla " (at least now is showing error at page)

2017/12/03 14:39:41 [error] 6907#6907: *41738 FastCGI sent in stderr: "PHP message: PHP Warning: mysqli_select_db() expects exactly 2 parameters, 1 given in /var/www/wow.trollandia.it/armory/configuration/mysql.php on line 60
PHP message: PHP Warning: mysqli_error() expects exactly 1 parameter, 0 given in /var/www/wow.trollandia.it/armory/configuration/mysql.php on line 60" while reading response header from upstream, client: 1xxxxx, server: wow.trollandia.it, request: "GET /armory/ HTTP/1.1", upstream: "fastcgi://unix:/run/php/php7.0-fpm.sock:", host: "wow.trollandia.it"

my armory https://wow.trollandia.it/armory/

That error is caused by this:
mysqli_select_db($needed_db[3]) or die("Unable to connect to ".$db_type." DB of realm ".$realm_name.": ".mysqli_error());

@ryanschulze
Copy link

The mysqli_* functions typically expect you to pass which DB connection you want to use. I might have a look at this later this week.

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

No branches or pull requests

3 participants