avoiding markdown rendering when reading task list
Signed-off-by: Stephan Richter <s.richter@srsoftware.de>
This commit is contained in:
@@ -139,6 +139,7 @@ public class Field {
|
|||||||
|
|
||||||
public static final String RECEIVERS = "receivers";
|
public static final String RECEIVERS = "receivers";
|
||||||
public static final String REDIRECT = "redirect";
|
public static final String REDIRECT = "redirect";
|
||||||
|
public static final String RENDER = "render";
|
||||||
public static final String RENDERED = "rendered";
|
public static final String RENDERED = "rendered";
|
||||||
public static final String REQUIRED_TASKS_IDS = "required_tasks_ids";
|
public static final String REQUIRED_TASKS_IDS = "required_tasks_ids";
|
||||||
|
|
||||||
|
|||||||
@@ -118,8 +118,7 @@ public class Project extends UmbrellaObject {
|
|||||||
return tagColors;
|
return tagColors;
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
public Map<String, Object> toMap(boolean renderMarkdown){
|
||||||
public Map<String, Object> toMap() {
|
|
||||||
var memberMap = new HashMap<Long,Map<String,Object>>();
|
var memberMap = new HashMap<Long,Map<String,Object>>();
|
||||||
if (members != null) for (var entry : members.entrySet()){
|
if (members != null) for (var entry : members.entrySet()){
|
||||||
memberMap.put(entry.getKey(),entry.getValue().toMap());
|
memberMap.put(entry.getKey(),entry.getValue().toMap());
|
||||||
@@ -128,7 +127,7 @@ public class Project extends UmbrellaObject {
|
|||||||
for (var state : allowedStates) stateMap.put(state.code(),state.name());
|
for (var state : allowedStates) stateMap.put(state.code(),state.name());
|
||||||
return super.map(
|
return super.map(
|
||||||
NAME,name,
|
NAME,name,
|
||||||
DESCRIPTION,mapMarkdown(description),
|
DESCRIPTION, renderMarkdown ? mapMarkdown(description) : Map.of(SOURCE,description),
|
||||||
STATUS,status,
|
STATUS,status,
|
||||||
COMPANY_ID,companyId,
|
COMPANY_ID,companyId,
|
||||||
SHOW_CLOSED,showClosed,
|
SHOW_CLOSED,showClosed,
|
||||||
@@ -137,6 +136,11 @@ public class Project extends UmbrellaObject {
|
|||||||
TAG_COLORS,tagColors);
|
TAG_COLORS,tagColors);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public Map<String, Object> toMap() {
|
||||||
|
return toMap(true);
|
||||||
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public String toString() {
|
public String toString() {
|
||||||
return name();
|
return name();
|
||||||
|
|||||||
@@ -59,7 +59,7 @@
|
|||||||
}
|
}
|
||||||
|
|
||||||
async function loadProject(pid){
|
async function loadProject(pid){
|
||||||
const url = api(`project/${pid}`);
|
const url = api(`project/${pid}?render=false`);
|
||||||
const resp = await get(url);
|
const resp = await get(url);
|
||||||
if (resp.ok){
|
if (resp.ok){
|
||||||
projects[pid] = await resp.json();
|
projects[pid] = await resp.json();
|
||||||
@@ -67,7 +67,7 @@
|
|||||||
}
|
}
|
||||||
|
|
||||||
async function load(){
|
async function load(){
|
||||||
const url = api(`task?offset=${bounds.offset}&limit=${bounds.limit}`);
|
const url = api(`task?offset=${bounds.offset}&limit=${bounds.limit}&render=false`);
|
||||||
const resp = await get(url);
|
const resp = await get(url);
|
||||||
if (resp.ok){
|
if (resp.ok){
|
||||||
let newTasks = await resp.json();
|
let newTasks = await resp.json();
|
||||||
|
|||||||
@@ -144,9 +144,11 @@ public class ProjectModule extends BaseHandler implements ProjectService {
|
|||||||
}
|
}
|
||||||
|
|
||||||
private boolean getProject(HttpExchange ex, long projectId, UmbrellaUser user) throws IOException, UmbrellaException {
|
private boolean getProject(HttpExchange ex, long projectId, UmbrellaUser user) throws IOException, UmbrellaException {
|
||||||
|
var params = queryParam(ex);
|
||||||
|
var render = !(params.get(RENDER) instanceof String r) || r.equals("true");
|
||||||
var project = loadMembers(projectDb.load(projectId));
|
var project = loadMembers(projectDb.load(projectId));
|
||||||
if (!project.hasMember(user)) throw notAmember(t(PROJECT_WITH_ID,ID,project.name()));
|
if (!project.hasMember(user)) throw notAmember(t(PROJECT_WITH_ID,ID,project.name()));
|
||||||
var map = project.toMap();
|
var map = project.toMap(render);
|
||||||
project.companyId().map(companyService()::get).map(Company::toMap).ifPresent(data -> map.put(Field.COMPANY,data));
|
project.companyId().map(companyService()::get).map(Company::toMap).ifPresent(data -> map.put(Field.COMPANY,data));
|
||||||
return sendContent(ex,map);
|
return sendContent(ex,map);
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -280,8 +280,9 @@ public class TaskModule extends BaseHandler implements TaskService {
|
|||||||
} catch (NumberFormatException e) {
|
} catch (NumberFormatException e) {
|
||||||
throw invalidField(LIMIT, t(Text.NUMBER));
|
throw invalidField(LIMIT, t(Text.NUMBER));
|
||||||
}
|
}
|
||||||
|
var render = !(params.get(RENDER) instanceof String r) || r.equals("true");
|
||||||
var tasks = taskDb.listUserTasks(user.id(), limit, offset, false);
|
var tasks = taskDb.listUserTasks(user.id(), limit, offset, false);
|
||||||
var mapped = loadMembers(tasks).stream().map(Task::toMap);
|
var mapped = loadMembers(tasks).stream().map(t -> t.toMap(render));
|
||||||
return sendContent(ex, mapped);
|
return sendContent(ex, mapped);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user