search

thinker007 / google-refine

Automatically exported from code.google.com/p/google-refine
0 stars 0 forks source link

Hangs when setting cell value to large JSON string #135

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. Edit any text cell
2. Paste the JSON below into the edit box (there's nothing special about it 
though, it will happen with any reasonably complex JSON).
3. Press Apply.

What is the expected output? What do you see instead?

Expect the value to be saved.  Instead, it shows the "Working" lightbox/gif 
 which never goes away - you have to refresh the page.  The edit does not succeed.  

Seems like a JSON encoding problem.

What version of the product are you using? On what operating system?

Gridworks development build, Windows 7.

Please provide any additional information below.

{
  "header": {
    "code": "1",
    "message": "Get Topic Successful"
  },
  "result": {
    "freebase_id": "/en/anthony_rapp",
    "topicuri": "anthony-rapp",
    "daylife_id": "0fbnexm7mJboV",
    "metadata": [
      {
        "name": "daylifeid",
        "value": "0fbnexm7mJboV",
        "source": "CrowdFusion"
      },
      {
        "name": "myspaceid",
        "value": "anthonyrapp",
        "source": "CrowdFusion"
      },
      {
        "name": "officialhomepage",
        "value": "http://anthonyrapp.com/",
        "source": "CrowdFusion"
      },
      {
        "name": "topicname",
        "value": "Anthony Rapp",
        "source": "CrowdFusion"
      },
      {
        "name": "freebaseimageid",
        "value": [
          "/m/02b8qt8",
          "/wikipedia/images/commons_id/2985706"
        ],
        "source": "Freebase"
      },
      {
        "name": "freebasemachineid",
        "value": "/m/03ysz4",
        "source": "Freebase"
      },
      {
        "name": "myspaceid",
        "value": "anthonyrapp",
        "source": "Freebase"
      },
      {
        "name": "officialhomepage",
        "value": "http://anthonyrapp.com/",
        "source": "Freebase"
      },
      {
        "name": "topicname",
        "value": "Anthony Rapp",
        "source": "Freebase"
      },
      {
        "name": "wikipediaurlid",
        "value": "1008352",
        "source": "Freebase"
      }
    ],
    "images": [
      {
        "size": "129x150",
        "url": "http://crowdfusion.myspacecdn.com/media/2010/06/16/anthony-rapp-150.jpg",
        "source": "CrowdFusion"
      },
      {
        "size": "140x140",
        "url": "http://crowdfusion.myspacecdn.com/media/2010/06/16/anthony-rapp-140x140.jpg",
        "source": "CrowdFusion"
      },
      {
        "size": "150x113",
        "url": "http://crowdfusion.myspacecdn.com/media/2010/06/16/anthony-rapp-150x113.jpg",
        "source": "CrowdFusion"
      },
      {
        "size": "150x150",
        "url": "http://crowdfusion.myspacecdn.com/media/2010/06/10/anthony-rapp-1-150.jpg",
        "source": "CrowdFusion"
      },
      {
        "size": "150x75",
        "url": "http://crowdfusion.myspacecdn.com/media/Anthony Rapp-150x75.jpg",
        "source": "CrowdFusion"
      },
      {
        "size": "200x300",
        "url": "http://crowdfusion.myspacecdn.com/media/Anthony Rapp-200x300.jpg",
        "source": "CrowdFusion"
      },
      {
        "size": "211x328",
        "url": "http://crowdfusion.myspacecdn.com/media/2010/06/16/anthony-rapp-211x328.jpg",
        "source": "CrowdFusion"
      },
      {
        "size": "300x211",
        "url": "http://crowdfusion.myspacecdn.com/media/2010/06/16/anthony-rapp-328x211.jpg",
        "source": "CrowdFusion"
      },
      {
        "size": "300x240",
        "url": "http://crowdfusion.myspacecdn.com/media/2010/06/16/anthony-rapp-520x240.jpg",
        "source": "CrowdFusion"
      },
      {
        "size": "300x322",
        "url": "http://crowdfusion.myspacecdn.com/media/Anthony Rapp-621x322.jpg",
        "source": "CrowdFusion"
      },
      {
        "size": "300x350",
        "url": "http://crowdfusion.myspacecdn.com/media/2010/06/16/anthony-rapp-300x351.jpg",
        "source": "CrowdFusion"
      },
      {
        "size": "300x351x",
        "url": "http://crowdfusion.myspacecdn.com/media/2010/06/16/anthony-rapp-300x351.jpg",
        "source": "CrowdFusion"
      },
      {
        "size": "55x55",
        "url": "http://crowdfusion.myspacecdn.com/media/2010/06/16/anthony-rapp-55x55.jpg",
        "source": "CrowdFusion"
      },
      {
        "size": "621x322x",
        "url": "http://crowdfusion.myspacecdn.com/media/Anthony Rapp-621x322.jpg",
        "source": "CrowdFusion"
      },
      {
        "size": "80x80",
        "url": "http://crowdfusion.myspacecdn.com/media/2010/06/16/anthony-rapp-80x80.jpg",
        "source": "CrowdFusion"
      }
    ],
    "topictype": [
      {
        "hub": "Celebrities"
      }
    ],
    "relatedtopics": [
      {
        "relatedtopicid": "/en/lucie_arnaz",
        "relation": "IsRelatedTo",
        "source": "Daylife"
      },
      {
        "relatedtopicid": "/en/mandy_patinkin",
        "relation": "IsRelatedTo",
        "source": "Daylife"
      },
      {
        "relatedtopicid": "/en/ben_affleck",
        "relation": "IsRelatedTo",
        "source": "Daylife"
      },
      {
        "relatedtopicid": "/en/bernadette_peters",
        "relation": "IsRelatedTo",
        "source": "Daylife"
      },
      {
        "relatedtopicid": "/en/peter_gallagher",
        "relation": "IsRelatedTo",
        "source": "CrowdFusion"
      },
      {
        "relatedtopicid": "/en/adam_pascal",
        "relation": "IsRelatedTo",
        "source": "CrowdFusion"
      },
      {
        "relatedtopicid": "/en/adam_pascal",
        "relation": "IsRelatedTo",
        "source": "CrowdFusion"
      },
      {
        "relatedtopicid": "/en/adam_pascal",
        "relation": "IsRelatedTo",
        "source": "Daylife"
      },
      {
        "relatedtopicid": "/en/john_stamos",
        "relation": "IsRelatedTo",
        "source": "CrowdFusion"
      },
      {
        "relatedtopicid": "/en/alan_cumming",
        "relation": "IsRelatedTo",
        "source": "CrowdFusion"
      },
      {
        "relatedtopicid": "/en/alan_cumming",
        "relation": "IsRelatedTo",
        "source": "CrowdFusion"
      },
      {
        "relatedtopicid": "/en/alan_cumming",
        "relation": "IsRelatedTo",
        "source": "Daylife"
      },
      {
        "relatedtopicid": "/en/anjelica_huston",
        "relation": "IsRelatedTo",
        "source": "Daylife"
      },
      {
        "relatedtopicid": "/en/elaine_stritch",
        "relation": "IsRelatedTo",
        "source": "Daylife"
      },
      {
        "relatedtopicid": "/en/elton_john",
        "relation": "IsRelatedTo",
        "source": "CrowdFusion"
      },
      {
        "relatedtopicid": "/en/ethel_waters",
        "relation": "IsRelatedTo",
        "source": "CrowdFusion"
      },
      {
        "relatedtopicid": "/en/hugh_jackman",
        "relation": "IsRelatedTo",
        "source": "Daylife"
      },
      {
        "relatedtopicid": "/en/idina_menzel",
        "relation": "IsRelatedTo",
        "source": "CrowdFusion"
      },
      {
        "relatedtopicid": "/en/idina_menzel",
        "relation": "IsRelatedTo",
        "source": "CrowdFusion"
      },
      {
        "relatedtopicid": "/en/idina_menzel",
        "relation": "IsRelatedTo",
        "source": "Daylife"
      },
      {
        "relatedtopicid": "/en/stephen_sondheim",
        "relation": "IsRelatedTo",
        "source": "Daylife"
      },
      {
        "relatedtopicid": "/en/joan_rivers",
        "relation": "IsRelatedTo",
        "source": "Daylife"
      },
      {
        "relatedtopicid": "/en/joel_grey",
        "relation": "IsRelatedTo",
        "source": "Daylife"
      },
      {
        "relatedtopicid": "/en/joey_lauren_adams",
        "relation": "IsRelatedTo",
        "source": "Daylife"
      },
      {
        "relatedtopicid": "/en/judith_light",
        "relation": "IsRelatedTo",
        "source": "Daylife"
      },
      {
        "relatedtopicid": "/en/liz_callaway",
        "relation": "IsRelatedTo",
        "source": "Daylife"
      },
      {
        "relatedtopicid": "/en/mark_wahlberg",
        "relation": "IsRelatedTo",
        "source": "CrowdFusion"
      },
      {
        "relatedtopicid": "/en/matthew_broderick",
        "relation": "IsRelatedTo",
        "source": "CrowdFusion"
      },
      {
        "relatedtopicid": "/en/michael_cerveris",
        "relation": "IsRelatedTo",
        "source": "Daylife"
      },
      {
        "relatedtopicid": "/en/parker_posey",
        "relation": "IsRelatedTo",
        "source": "Daylife"
      },
      {
        "relatedtopicid": "/en/philip_seymour_hoffman",
        "relation": "IsRelatedTo",
        "source": "Daylife"
      },
      {
        "relatedtopicid": "/en/suzanne_somers",
        "relation": "IsRelatedTo",
        "source": "CrowdFusion"
      },
      {
        "relatedtopicid": "/en/taye_diggs",
        "relation": "IsRelatedTo",
        "source": "Daylife"
      },
      {
        "relatedtopicid": "/en/tommy_lee",
        "relation": "IsRelatedTo",
        "source": "CrowdFusion"
      },
      {
        "relatedtopicid": "/en/tommy_lee",
        "relation": "IsRelatedTo",
        "source": "CrowdFusion"
      },
      {
        "relatedtopicid": "/en/tommy_lee",
        "relation": "IsRelatedTo",
        "source": "Daylife"
      }
    ]
  }
}

Original issue reported on code.google.com by wbl...@myspace-inc.com on 21 Sep 2010 at 6:20

GoogleCodeExporter commented 9 years ago 
This was SVN revision 1261, last change date 2010-09-09.

Original comment by wbl...@myspace-inc.com on 21 Sep 2010 at 6:30

GoogleCodeExporter commented 9 years ago 
Fixed by r1374.

Original comment by dfhu...@google.com on 28 Sep 2010 at 4:43

GoogleCodeExporter commented 9 years ago

Original comment by tfmorris on 18 Sep 2012 at 2:58