Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Setting hash fixed #58

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Conversation

jkorz
Copy link

@jkorz jkorz commented Feb 16, 2013

The latest commit fixes three issues I was having.

  1. When trying to add a task from the Sprint page, it would throw a javascript error saying that a tracker wasn't found.
  2. When trying to list tasks on the Task page, nothing would show up.
  3. When in the settings menu for AgileDwarf, the settings wouldn't stick after setting them.

I traced this down to the fact that the settings hash as stored in the Setting model had strings for the keys rather than symbols. When I changed all the keys from symbols to strings, the problems all went away.

My Environment:

Environment:
Redmine version 2.2.3.stable
Ruby version 1.9.2 (i686-linux)
Rails version 3.2.12
Environment production
Database adapter Mysql2
Redmine plugins:
AgileDwarf 0.0.3

codingjoe and others added 10 commits September 18, 2012 12:58
The input never got created, therefor you'd get a null pointer exception in line 5 which is now line 6.
$('<tag></tag>') is not just slower but doesn't allow any attributes.
This should be faster and work just fine in any browser.
It deletes the textPlaceholder() call. Input (jQuery Object) does not have this method. Besides that there is no reference on what this method does anyways.
This should work just fine.
This removes the call to textPlaceholer() on input (jQuery Document).
There is no method textPlaceholder() and I can't find any reference for it.
It seems to have no function but ends up to generate an exception.
@AlexAndrascu
Copy link

+1 To this. Tested and works.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants