Closed Bug 1044989 Opened 10 years ago Closed 6 years ago

package.js option of integer type inside a string breaks inline options of type `radio`

Categories

(Add-on SDK Graveyard :: General, defect, P2)

All
Windows 7
defect

Tracking

(Not tracked)

RESOLVED INCOMPLETE

People

(Reporter: alexufo, Unassigned)

References

Details

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:33.0) Gecko/20100101 Firefox/33.0 (Beta/Release) Build ID: 20140720030203 Steps to reproduce: "preferences": [ { "name": "dpi", "type": "radio", "title": "Точек на дюйм у принтера", "value": "200dpi", // "200" - only digits didnt work "options": [ { "value": "200dpi", // "200" - only digits didnt work "label": "200dpi" }, { "value": "300dpi", // "300" - only digits didnt work "label": "300dpi" } ] } ] Actual results: If i want to set value as digit -this option didnt saved. Expected results: Value should be saved as digits too.
OS: All → Windows 7
(In reply to Alexufo from comment #0) > "value": "200dpi", // "200" - only digits didnt work you can use only digits, but _without_ quotes, and this should work. --- that said, it looks like we indeed have a bug here, but it's probably in the inline options code, not in the SDK code. with the package.json posted above, our code, correctly, creates a string pref, while the code on the addon options page tries to setIntPref, and naturally fails.
Flags: needinfo?(tomica+amo)
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: package.js option → package.js option of integer type inside a string breaks inline options of type `radio`
We should either try to fix about:addons page so it interprets types same as SDK does, or change how SDK understands types. zombie or erikvold would one of you mind to take this bug ? Thanks
Flags: needinfo?(evold)
Priority: -- → P2
I'd like to take a look but I'm not sure I'll be able to, and I know I can't until the end of my pto on oct 13th
Flags: needinfo?(evold)
Whiteboard: [good first bug]
I'm a new contributor, I'd like to take this bug up, can I get some more direction? Since it doesn't have a mentor, I am not sure who to ask.
Flags: needinfo?(rFobic)
Flags: needinfo?(erikvvold)
Because of the difficulty finding mentors and the expected life span of the SDK, we are removing [good first bug] from remaining SDK bugs.
Whiteboard: [good first bug]
Flags: needinfo?(rFobic)
Add-on SDK is no longer supported so resolving bugs as INCOMPLETE
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INCOMPLETE
Flags: needinfo?(erikvvold)
You need to log in before you can comment on or make changes to this bug.