{"action":"created","issue":{"url":"https://api.github.com/repos/jbosstools/jbosstools-openshift/issues/1186","repository_url":"https://api.github.com/repos/jbosstools/jbosstools-openshift","labels_url":"https://api.github.com/repos/jbosstools/jbosstools-openshift/issues/1186/labels{/name}","comments_url":"https://api.github.com/repos/jbosstools/jbosstools-openshift/issues/1186/comments","events_url":"https://api.github.com/repos/jbosstools/jbosstools-openshift/issues/1186/events","html_url":"https://github.com/jbosstools/jbosstools-openshift/pull/1186","id":156857455,"number":1186,"title":"[JBIDE 21068] Set maximum length for display name for project wizard","user":{"login":"bdshadow","id":5796973,"avatar_url":"https://avatars.githubusercontent.com/u/5796973?v=3","gravatar_id":"","url":"https://api.github.com/users/bdshadow","html_url":"https://github.com/bdshadow","followers_url":"https://api.github.com/users/bdshadow/followers","following_url":"https://api.github.com/users/bdshadow/following{/other_user}","gists_url":"https://api.github.com/users/bdshadow/gists{/gist_id}","starred_url":"https://api.github.com/users/bdshadow/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bdshadow/subscriptions","organizations_url":"https://api.github.com/users/bdshadow/orgs","repos_url":"https://api.github.com/users/bdshadow/repos","events_url":"https://api.github.com/users/bdshadow/events{/privacy}","received_events_url":"https://api.github.com/users/bdshadow/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2016-05-25T21:44:09Z","updated_at":"2016-05-26T19:10:54Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/jbosstools/jbosstools-openshift/pulls/1186","html_url":"https://github.com/jbosstools/jbosstools-openshift/pull/1186","diff_url":"https://github.com/jbosstools/jbosstools-openshift/pull/1186.diff","patch_url":"https://github.com/jbosstools/jbosstools-openshift/pull/1186.patch"},"body":"The limit is, actually, set in swt Text and it is [platform dependent](http://help.eclipse.org/luna/topic/org.eclipse.platform.doc.isv/reference/api/org/eclipse/swt/widgets/Text.html#LIMIT). In my Fedora 23 \r\n```\r\npublic int getTextLimit () {\r\n\tcheckWidget ();\r\n\tif ((style & SWT.MULTI) != 0) return LIMIT;\r\n\tint limit = OS.gtk_entry_get_max_length (handle);\r\n\treturn limit == 0 ? 0xFFFF : untranslateOffset (limit);\r\n}\r\n```\r\nalways returns 0xFFFF which is exactly 65536. So couldn't test it in a usual way, but just setting the limit lower for testing purposes.\r\nHowever, i added ValidationStatus.error when input is bigger so that this check works in another OSs."},"comment":{"url":"https://api.github.com/repos/jbosstools/jbosstools-openshift/issues/comments/221965859","html_url":"https://github.com/jbosstools/jbosstools-openshift/pull/1186#issuecomment-221965859","issue_url":"https://api.github.com/repos/jbosstools/jbosstools-openshift/issues/1186","id":221965859,"user":{"login":"fbricon","id":148698,"avatar_url":"https://avatars.githubusercontent.com/u/148698?v=3","gravatar_id":"","url":"https://api.github.com/users/fbricon","html_url":"https://github.com/fbricon","followers_url":"https://api.github.com/users/fbricon/followers","following_url":"https://api.github.com/users/fbricon/following{/other_user}","gists_url":"https://api.github.com/users/fbricon/gists{/gist_id}","starred_url":"https://api.github.com/users/fbricon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fbricon/subscriptions","organizations_url":"https://api.github.com/users/fbricon/orgs","repos_url":"https://api.github.com/users/fbricon/repos","events_url":"https://api.github.com/users/fbricon/events{/privacy}","received_events_url":"https://api.github.com/users/fbricon/received_events","type":"User","site_admin":false},"created_at":"2016-05-26T19:10:54Z","updated_at":"2016-05-26T19:10:54Z","body":"Aside from a couple minuscule issues, LGTM. Will merge once @bdshadow updates the PR"}}
{"actor":{"display_login":"fbricon"}}