using enter instead of click on "Add" when adding competency breaks

Bug #194839 reported by Filip Sufitchi
4
Affects Status Importance Assigned to Milestone
CanDo
Low
Chris Carey

Bug Description

While using the treeview for competency management to create a new competency, after the form is filled out, hitting enter spawns an error instead of creating a competency. Clicking "Add" however works. This is the server error I'm getting:

/home/fsufitch/cando-trunk/eggs/zope.app.form-3.4.1-py2.4.egg/zope/app/form/browser/widget.py:537: DeprecationWarning: None was passed for attribute 'href'. Passing None as attribute values to renderTag is deprecated. Passing None as an attribute value will be disallowed starting in Zope 3.3.
  return renderTag(tag, **kw) + " />"

Is the form being submitted right?

Jason Straw (jasonstraw)
Changed in cando:
assignee: nobody → chris-carey
importance: Undecided → Low
status: New → Triaged
Revision history for this message
Chris Carey (chris-carey) wrote :

The problem that is occuring here is that in the tree view of competencies, the widget for displaying the competency's global id is failing, because the competency doesn't have a global id. The current patch I did was simply not displaying the global id widget in virginiacompetencytreenode.pt. When global ids are set, this can be removed. However, I'm not committing yet unless this is approved.

In VirginiaCompetency, globalId=None by default. I'm not familiar with how global ids are set, but there either needs to be a better default than None, or a check to see if the competency's global id is None or not. At the page template level, I can't check whether the competency has a global id or not.

If someone knows how/where I can perform this check, please let me know.

Revision history for this message
Filip Sufitchi (fsufitch) wrote :

Bump this bug to be checked for validity since globalIds are central to competencies now and I don't believe they default to None anymore.

Revision history for this message
Jason Straw (jasonstraw) wrote :

Obsolete. 251259 removes this ability.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers