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.