/* type is borrowed */
} else
eWarning("eListboxPythonConfigContent: second value of tuple is not a tuple.");
- /* value is borrowed */
+ if (value)
+ Py_DECREF(value);
}
if (selected && (!local_style || !local_style->m_selection))
style.setStyle(painter, eWindowStyle::styleListboxNormal);
if (pbackColor)
{
- int color = PyInt_AsLong(pbackColor);
+ unsigned int color = PyInt_AsUnsignedLongMask(pbackColor);
painter.setBackgroundColor(gRGB(color));
} // transparent background?
// if we have a local background color set, use that.
style.setStyle(painter, eWindowStyle::styleListboxSelected);
if (pbackColorSelected)
{
- int color = PyInt_AsLong(pbackColorSelected);
+ unsigned int color = PyInt_AsUnsignedLongMask(pbackColorSelected);
painter.setBackgroundColor(gRGB(color));
}
else if (local_style && local_style->m_background_color_selected_set)
style.setStyle(painter, eWindowStyle::styleListboxSelected);
if (pbackColorSelected)
{
- int color = PyInt_AsLong(pbackColorSelected);
+ unsigned int color = PyInt_AsUnsignedLongMask(pbackColorSelected);
painter.setBackgroundColor(gRGB(color));
}
else if (local_style && local_style->m_background_color_selected_set)
style.setStyle(painter, eWindowStyle::styleListboxNormal);
if (pbackColor)
{
- int color = PyInt_AsLong(pbackColor);
+ unsigned int color = PyInt_AsUnsignedLongMask(pbackColor);
painter.setBackgroundColor(gRGB(color));
}/* if we have a local background color set, use that. */
else if (local_style && local_style->m_background_color_set)
{
if (pforeColorSelected)
{
- int color = PyInt_AsLong(pforeColorSelected);
+ unsigned int color = PyInt_AsUnsignedLongMask(pforeColorSelected);
painter.setForegroundColor(gRGB(color));
}
/* if we have a local foreground color set, use that. */
{
if (pforeColor)
{
- int color = PyInt_AsLong(pforeColor);
+ unsigned int color = PyInt_AsUnsignedLongMask(pforeColor);
painter.setForegroundColor(gRGB(color));
}
/* if we have a local foreground color set, use that. */
}
}
+static ePyObject lookupColor(ePyObject color, ePyObject data)
+{
+ if (color == Py_None)
+ return ePyObject();
+
+ if ((!color) && (!data))
+ return color;
+
+ unsigned int icolor = PyInt_AsUnsignedLongMask(color);
+
+ /* check if we have the "magic" template color */
+ if ((icolor & 0xFF000000) == 0xFF000000)
+ {
+ int index = icolor & 0xFFFFFF;
+ eDebug("[eListboxPythonMultiContent] template color index: %d", index);
+ return PyTuple_GetItem(data, index);
+ }
+
+ if (color == Py_None)
+ return ePyObject();
+
+ return color;
+}
+
void eListboxPythonMultiContent::paint(gPainter &painter, eWindowStyle &style, const ePoint &offset, int selected)
{
gRegion itemregion(eRect(offset, m_itemsize));
painter.clip(itemregion);
clearRegion(painter, style, local_style, ePyObject(), ePyObject(), ePyObject(), ePyObject(), selected, itemregion, sel_clip);
- ePyObject items;
+ ePyObject items, buildfunc_ret;
if (m_list && cursorValid())
{
+ /* a multicontent list can be used in two ways:
+ either each item is a list of (TYPE,...)-tuples,
+ or there is a template defined, which is a list of (TYPE,...)-tuples,
+ and the list is an unformatted tuple. The template then references items from the list.
+ */
items = PyList_GET_ITEM(m_list, m_cursor); // borrowed reference!
if (m_buildFunc)
if (PyCallable_Check(m_buildFunc)) // when we have a buildFunc then call it
{
if (PyTuple_Check(items))
- items = PyObject_CallObject(m_buildFunc, items);
+ buildfunc_ret = items = PyObject_CallObject(m_buildFunc, items);
else
eDebug("items is no tuple");
}
goto error_out;
}
- if (!PyList_Check(items))
+ if (!m_template)
{
- eDebug("eListboxPythonMultiContent: list entry %d is not a list", m_cursor);
- goto error_out;
+ if (!PyList_Check(items))
+ {
+ eDebug("eListboxPythonMultiContent: list entry %d is not a list (non-templated)", m_cursor);
+ goto error_out;
+ }
+ } else
+ {
+ if (!PyTuple_Check(items))
+ {
+ eDebug("eListboxPythonMultiContent: list entry %d is not a tuple (templated)", m_cursor);
+ goto error_out;
+ }
+ }
+
+ ePyObject data;
+
+ /* if we have a template, use the template for the actual formatting.
+ we will later detect that "data" is present, and refer to that, instead
+ of the immediate value. */
+ int start = 1;
+ if (m_template)
+ {
+ data = items;
+ items = m_template;
+ start = 0;
}
int size = PyList_Size(items);
- for (int i = 1; i < size; ++i)
+ for (int i = start; i < size; ++i)
{
ePyObject item = PyList_GET_ITEM(items, i); // borrowed reference!
}
if (size > 8)
- {
- pforeColor = PyTuple_GET_ITEM(item, 8);
- if (pforeColor == Py_None)
- pforeColor=ePyObject();
- }
+ pforeColor = lookupColor(PyTuple_GET_ITEM(item, 8), data);
+
if (size > 9)
- {
- pforeColorSelected = PyTuple_GET_ITEM(item, 9);
- if (pforeColorSelected == Py_None)
- pforeColorSelected=ePyObject();
- }
+ pforeColorSelected = lookupColor(PyTuple_GET_ITEM(item, 9), data);
+
if (size > 10)
- {
- pbackColor = PyTuple_GET_ITEM(item, 10);
- if (pbackColor == Py_None)
- pbackColor=ePyObject();
- }
+ pbackColor = lookupColor(PyTuple_GET_ITEM(item, 10), data);
+
if (size > 11)
- {
- pbackColorSelected = PyTuple_GET_ITEM(item, 11);
- if (pbackColorSelected == Py_None)
- pbackColorSelected=ePyObject();
- }
+ pbackColorSelected = lookupColor(PyTuple_GET_ITEM(item, 11), data);
+
if (size > 12)
{
pborderWidth = PyTuple_GET_ITEM(item, 12);
pborderWidth=ePyObject();
}
if (size > 13)
- {
- pborderColor = PyTuple_GET_ITEM(item, 13);
- if (pborderColor == Py_None)
- pborderColor=ePyObject();
- }
+ pborderColor = lookupColor(PyTuple_GET_ITEM(item, 13), data);
+
+ if (PyInt_Check(pstring) && data) /* if the string is in fact a number, it refers to the 'data' list. */
+ pstring = PyTuple_GetItem(data, PyInt_AsLong(pstring));
+
+ /* don't do anything if we have 'None' as string */
+ if (pstring == Py_None)
+ continue;
const char *string = (PyString_Check(pstring)) ? PyString_AsString(pstring) : "<not-a-string>";
int x = PyInt_AsLong(px) + offset.x();
painter.clip(rect);
if (pborderColor)
{
- int color = PyInt_AsLong(pborderColor);
+ unsigned int color = PyInt_AsUnsignedLongMask(pborderColor);
painter.setForegroundColor(gRGB(color));
}
int width = PyInt_AsLong(pwidth);
int height = PyInt_AsLong(pheight);
int filled = PyInt_AsLong(pfilled_perc);
+
+ if ((filled < 0) && data) /* if the string is in a negative number, it refers to the 'data' list. */
+ filled = PyInt_AsLong(PyTuple_GetItem(data, -filled));
+
int bwidth = pborderWidth ? PyInt_AsLong(pborderWidth) : 2;
eRect rect(x, y, width, height);
goto error_out;
}
+ if (PyInt_Check(ppixmap) && data) /* if the pixemap is in fact a number, it refers to the 'data' list. */
+ ppixmap = PyTuple_GetItem(data, PyInt_AsLong(ppixmap));
+
+ /* don't do anything if we have 'None' as pixmap */
+ if (ppixmap == Py_None)
+ continue;
+
int x = PyInt_AsLong(px) + offset.x();
int y = PyInt_AsLong(py) + offset.y();
int width = PyInt_AsLong(pwidth);
}
if (size > 6)
- {
- pbackColor = PyTuple_GET_ITEM(item, 6);
- if (pbackColor == Py_None)
- pbackColor=ePyObject();
- }
+ pbackColor = lookupColor(PyTuple_GET_ITEM(item, 6), data);
+
if (size > 7)
- {
- pbackColorSelected = PyTuple_GET_ITEM(item, 7);
- if (pbackColorSelected == Py_None)
- pbackColorSelected=ePyObject();
- }
+ pbackColorSelected = lookupColor(PyTuple_GET_ITEM(item, 7), data);
eRect rect(x, y, width, height);
painter.clip(rect);
style.drawFrame(painter, eRect(offset, m_itemsize), eWindowStyle::frameListboxEntry);
error_out:
- if (m_buildFunc && PyCallable_Check(m_buildFunc) && items)
- Py_DECREF(items);
+ if (buildfunc_ret)
+ Py_DECREF(buildfunc_ret);
painter.clippop();
}
{
ePyObject ret = PyObject_CallObject(m_selectableFunc, args);
if (ret)
- return ret == Py_True;
+ {
+ bool retval = ret == Py_True;
+ Py_DECREF(ret);
+ return ret;
+ }
eDebug("call m_selectableFunc failed!!! assume not callable");
}
else
item = PyList_GET_ITEM(item, 0);
if (item != Py_None)
return 1;
+ } else if (PyTuple_Check(item))
+ {
+ item = PyTuple_GET_ITEM(item, 0);
+ if (item != Py_None)
+ return 1;
}
else if (m_buildFunc && PyCallable_Check(m_buildFunc))
return 1;
if (m_listbox)
m_listbox->entryRemoved(idx);
}
+
+void eListboxPythonMultiContent::setTemplate(ePyObject tmplate)
+{
+ m_template = tmplate;
+}