forked from Sarjuuk/aowow
-
Notifications
You must be signed in to change notification settings - Fork 33
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
chore. Update the ac compatible repository #53
Open
pangolp
wants to merge
254
commits into
azerothcore:master
Choose a base branch
from
pangolp:update
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* use strings from globalstrings.lua to format time in item and spell tooltips * fixed item filter ItemCooldown * fixed timeAgo format * move item duration to correct position in tooltip
* get auras from creature_template_addon for tabs - npc: abilities - spell: used by npc
* fixed broken link to required event
* fix icon order for pet talents
* fixed rarity color for spells that create items
* fix text vars in tooltips for missing entities * fix notice in weapon GS calculation * reduce tooltip max width slightly to prevent overlap with secondary tooltip
* fixed effect index offset on DetailPage
* disable compare button for empty item sets * do not display summary for empty item sets
* move localized option sorting to postCache() to prevent real order display in cached versions
* append a filemtime timestamp to js/css files to work around browser caching after update * shuffle script handling around a bit * also user pages cant have community content * also fix breadcrumbs on items page
* do not prefil search form with unchecked user input * thx @Endalaust
* fixed missing icons in used-by-spell tab
* fix tooltip localization after 9f1cbc0
* small typo
* do not skip generic page constructor ... ever
* log approver / time as intended
* display class/spec in category if applicable * make class/spec searchable * unify class/spec display with tooltips
* fixed more error pages without styles/scripts * fixed sourcemore for items if more then 300 items where requested at once * fixed item name localization in rare cases
* convert and assign correct icons for honor points and arena points from /Interface/PvPFrame
* resolve more col name conflicts .. should probably rename the cols in the table itself...
* sanity check slot / invtype realtion on setup (and hide offenders) * hide internal/unused items by name part * always truncate table to get rid of old data * fixing one->many relation revealed by replacing REPLACE with INSERT in creature setup
* also fix width for secondary tooltips
* remove padding from profiles icon list
* fix syntax of db_structure.sql * not to self: SHOW CREATE TABLE doesn't return a terminated statement * fixes Sarjuuk#431
* fix confusion of SpellClassMask fields * no functional change
* reapply lost /4 modifier for enchantment scores
* removed old workaround for js date object creation
* kill an antique typo causing an error whenever a tooltip for a hearthstoneesque effect is displayed for Locale FR an RU
* also log POST data
* fixed resolving procSpells with a triggerSpell of 0 * spells are now always displayed in listview
* fixed help text referencing renamed parameter
* avoid using "echo" to write to CLI as php mistakes it for sent headers (see php-src #12303) * as we are using fwrite now, errors are written to STDERR instead of STDOUT * fixes an issue where reloading the config would cause "ini_set(): Session ini settings cannot be changed after headers have already been sent" to be spammed
* fixed several deprecation notices and warnings from MySQL8, most notably: - SQL_CALC_FOUND_ROWS: stopped using DBSimple::selectPage and query 'SELECT COUNT(*) ...' separately where needed - ON DUPLICATE KEY UPDATE ... VALUES(): use row alias for new values instead of VALUES function - boolean shorthands to long form (&& -> AND, etc)
* don't use 'delay' to calc added dps from ammunition
* do not display crafted item tooltip inside recipe tooltip if the craft spell doesn't create an item. (notably: enchantments were displaying enchantment scrolls) * always display reagent cost if set
* attempted cleanup in effect generation and template * display summon properties * display unit of effect value * fix spell effect layout, typos, missing tools, ...
* changed logic to not display required arena rating in tooltip or infobox if there are sources that don't require it. * also, should there be a difference in sources, display lowest required rating, not not highest.
* fix usage of generic search wildcards: ? * * only transform search form fields expected to be text * some cleanup (magic numbers to define, fn return types, nullsafe assignemnts)
* fix spawn point calculation on instance maps without dungeonmap entry (can't wait to see what breaks elsewhere)
* even more forgotten renames in help text * see 07e001e
* fix empty scaling bar * fix setting max level for scaling tooltips * fix displayed periodic power gain value for runic power and rage * attempt to fix spell coefficient calculation (dear god...) * append % sign to value for two more auras * display value per combo point
* fix invalid type error * simplyfy query building (there are no longer tables with an index named `entry`)
* fix totals query params after 481a3dc
* fix totals query. Rather treat original query as subquery than trying to modify it * fixes queries utilizing HAVING and GROUP BY
* reports can have 'null' subject (general bug reports, general feedback)
* fix icon formater: num/qty can be strings
* filter empty tabs before trying to display, case in point * do not always assign conditions tab if empty
* fix invalid return type in createConditionsForWeights()
* fix invalid return type in handleWeightscales() for action: delete
* fix index error in ArenaTeam member update query
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I clarify that I cannot test the code, because I do not have a local compilation of this repository. Later, I will try to have one, but for the moment, I do not have it.
closes wowgaming#12
closes #47
it could close also #40