validator needs to look at other fields from compare-locales

RESOLVED FIXED in 5.2

Status

Product:
addons.mozilla.org Graveyard
Component:
Developer Pages
Type:
defect
Status:
RESOLVED FIXED
Reported:
10 years ago
Modified:
3 years ago

People

(Reporter: clouserw, Assigned: adriank)

Tracking

Version:
unspecified

Details

Attachments

(1 attachment)

add check for missing files and missing entities in them
1.43 KB, patch
clouserw
: review+
Details | Diff | Splinter Review
Reporter

Description

•
10 years ago 
On line 668 of validation.php AMO needs to look for "missingFiles" and "missingEntitiesInMissingFiles" as well.
Assignee

Comment 1

•
10 years ago 
adds checking for missing files and missing entities in them, changes the description for "missingEntities" to note that they are missing in existing files
Attachment #405955 - Flags: review?(clouserw)
 Reporter

Updated

•
10 years ago
Attachment #405955 - Flags: review?(clouserw) → review+
 Reporter

Comment 2

•
10 years ago 
I changed the wording around to make it easier for L10n but the patch is good.  Thanks, r53300
Assignee: nobody → adriank
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
 Reporter

Updated

•
10 years ago
Target Milestone: --- → 5.2
 Assignee

Updated

•
10 years ago
Assignee: adriank → akalla
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.