Closed Bug 741841 Opened 10 years ago Closed 10 years ago

Including a version range in a Plugin block shouldn't automatically add a targetApplication

Categories

(Toolkit :: Blocklist Policy Requests, defect, P1)

defect

Tracking

()

RESOLVED FIXED

People

(Reporter: jorgev, Assigned: cvan)

References

()

Details

I created this test block:
https://addons-dev.allizom.org/en-US/admin/models/blocklist/blocklistdetail/83/

Note the Blocklist Plugins section with min and max version. Also note I didn't specify a targetApplication

The downloaded blocklist item ends up being this:

> <pluginItem  blockID="p83">
>   <match name="filename" exp="(JavaPlugin2_NPAPI\.plugin)|(JavaAppletPlugin\.plugin)"
>   />
>   <versionRange  severity="1">
>     <targetApplication id="%7Bec8030f7-c20a-464f-9b0e-13a3a9e97384%7D">
>       <versionRange  minVersion="0" maxVersion="14.1.0" />
>     </targetApplication>
>   </versionRange>
> </pluginItem>

What I expected was this:

> <pluginItem  blockID="p83">
>   <match name="filename" exp="(JavaPlugin2_NPAPI\.plugin)|(JavaAppletPlugin\.plugin)"
>   />
>   <versionRange  severity="1" minVersion="0" maxVersion="14.1.0">
>   </versionRange>
> </pluginItem>

There should be no target application, and the min and max versions should apply to the parent versionRange node.
Summary: Including a version range in a Plugin block shouldn't automatically add an app id → Including a version range in a Plugin block shouldn't automatically add a targetApplication
Assignee: nobody → cvan
https://github.com/mozilla/zamboni/commit/1fc8f15
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Product: addons.mozilla.org → Toolkit
You need to log in before you can comment on or make changes to this bug.