Improved performance of large object tables.

This commit is contained in:
Martin Edenhofer 2015-11-20 10:47:41 +01:00
parent 8800b6b61a
commit 9447a08e6e
4 changed files with 71 additions and 58 deletions

View file

@ -264,6 +264,7 @@ class App.ControllerTable extends App.Controller
table = App.view('generic/table')( table = App.view('generic/table')(
table_id: @table_id table_id: @table_id
header: @headers header: @headers
attributes: attributes
objects: @objects objects: @objects
checkbox: @checkbox checkbox: @checkbox
radio: @radio radio: @radio

View file

@ -1,6 +1,5 @@
# Copyright (C) 2012-2014 Zammad Foundation, http://zammad-foundation.org/ # Copyright (C) 2012-2014 Zammad Foundation, http://zammad-foundation.org/
#= require_self #= require_self
#= require_tree ./lib/app_init #= require_tree ./lib/app_init
#= require ./config.coffee #= require ./config.coffee
@ -10,7 +9,8 @@
#= require_tree ./lib/app_post #= require_tree ./lib/app_post
class App extends Spine.Controller class App extends Spine.Controller
@viewPrint: (object, attribute_name) -> @viewPrint: (object, attribute_name, attributes) ->
if !attributes
attributes = {} attributes = {}
if object.constructor.attributesGet if object.constructor.attributesGet
attributes = object.constructor.attributesGet() attributes = object.constructor.attributesGet()
@ -34,11 +34,9 @@ class App extends Spine.Controller
if parts[0] && parts[1] && object[ parts[0] ] if parts[0] && parts[1] && object[ parts[0] ]
value = object[ parts[0] ][ parts[1] ] value = object[ parts[0] ][ parts[1] ]
#console.log('Pa', attribute_name, object, attribute_config, object[attribute_name], valueRef, value)
# if we have no config, get output this way # if we have no config, get output this way
if !attribute_config if !attribute_config
return @viewPrintItem( value ) return @viewPrintItem(value)
# check if valueRef already exists, no lookup needed later # check if valueRef already exists, no lookup needed later
if !valueRef if !valueRef
@ -48,10 +46,10 @@ class App extends Spine.Controller
if object[attribute_name_without_ref] if object[attribute_name_without_ref]
valueRef = object[attribute_name_without_ref] valueRef = object[attribute_name_without_ref]
return @viewPrintItem( value, attribute_config, valueRef ) @viewPrintItem(value, attribute_config, valueRef)
# define print name helper # define print name helper
@viewPrintItem: ( item, attribute_config = {}, valueRef ) -> @viewPrintItem: (item, attribute_config = {}, valueRef) ->
return '-' if item is undefined return '-' if item is undefined
return '-' if item is '' return '-' if item is ''
return item if !item return item if !item
@ -77,13 +75,13 @@ class App extends Spine.Controller
# execute callback on content # execute callback on content
if attribute_config.callback if attribute_config.callback
result = attribute_config.callback( result, attribute_config ) result = attribute_config.callback(result, attribute_config)
# text2html in textarea view # text2html in textarea view
isHtmlEscape = false isHtmlEscape = false
if attribute_config.tag is 'textarea' if attribute_config.tag is 'textarea'
isHtmlEscape = true isHtmlEscape = true
result = App.Utils.text2html( result ) result = App.Utils.text2html(result)
# remember, html snippets are already escaped # remember, html snippets are already escaped
else if attribute_config.tag is 'richtext' else if attribute_config.tag is 'richtext'
@ -95,9 +93,9 @@ class App extends Spine.Controller
result = attribute_config.options[result] result = attribute_config.options[result]
# translate content # translate content
if attribute_config.translate || ( isObject && item.translate && item.translate() ) if attribute_config.translate || (isObject && item.translate && item.translate())
isHtmlEscape = true isHtmlEscape = true
result = App.i18n.translateContent( result ) result = App.i18n.translateContent(result)
# transform date # transform date
if attribute_config.tag is 'date' if attribute_config.tag is 'date'
@ -121,71 +119,71 @@ class App extends Spine.Controller
result result
@view: (name) -> @view: (name) ->
template = ( params = {} ) -> template = (params = {}) ->
# define print name helper # define print name helper
params.P = ( object, attribute_name ) -> params.P = (object, attribute_name, attributes) ->
App.viewPrint( object, attribute_name ) App.viewPrint(object, attribute_name, attributes)
# define date format helper # define date format helper
params.date = ( time ) -> params.date = (time) ->
return '' if !time return '' if !time
timeObject = new Date(time) timeObject = new Date(time)
d = App.Utils.formatTime( timeObject.getDate(), 2 ) d = App.Utils.formatTime(timeObject.getDate(), 2)
m = App.Utils.formatTime( timeObject.getMonth() + 1, 2 ) m = App.Utils.formatTime(timeObject.getMonth() + 1, 2)
y = timeObject.getFullYear() y = timeObject.getFullYear()
"#{y}-#{m}-#{d}" "#{y}-#{m}-#{d}"
# define datetime format helper # define datetime format helper
params.datetime = ( time ) -> params.datetime = (time) ->
return '' if !time return '' if !time
timeObject = new Date(time) timeObject = new Date(time)
d = App.Utils.formatTime( timeObject.getDate(), 2 ) d = App.Utils.formatTime(timeObject.getDate(), 2)
m = App.Utils.formatTime( timeObject.getMonth() + 1, 2 ) m = App.Utils.formatTime(timeObject.getMonth() + 1, 2)
y = timeObject.getFullYear() y = timeObject.getFullYear()
S = App.Utils.formatTime( timeObject.getSeconds(), 2 ) S = App.Utils.formatTime(timeObject.getSeconds(), 2)
M = App.Utils.formatTime( timeObject.getMinutes(), 2 ) M = App.Utils.formatTime(timeObject.getMinutes(), 2)
H = App.Utils.formatTime( timeObject.getHours(), 2 ) H = App.Utils.formatTime(timeObject.getHours(), 2)
"#{y}-#{m}-#{d} #{H}:#{M}:#{S}" "#{y}-#{m}-#{d} #{H}:#{M}:#{S}"
# define decimal format helper # define decimal format helper
params.decimal = ( data, positions = 2 ) -> params.decimal = (data, positions = 2) ->
App.Utils.decimal(data, positions) App.Utils.decimal(data, positions)
# define translation helper # define translation helper
params.T = ( item, args... ) -> params.T = (item, args...) ->
App.i18n.translateContent( item, args... ) App.i18n.translateContent(item, args...)
# define translation inline helper # define translation inline helper
params.Ti = ( item, args... ) -> params.Ti = (item, args...) ->
App.i18n.translateInline( item, args... ) App.i18n.translateInline(item, args...)
# define translation for date helper # define translation for date helper
params.Tdate = ( item, args... ) -> params.Tdate = (item, args...) ->
App.i18n.translateDate( item, args... ) App.i18n.translateDate(item, args...)
# define translation for timestamp helper # define translation for timestamp helper
params.Ttimestamp = ( item, args... ) -> params.Ttimestamp = (item, args...) ->
App.i18n.translateTimestamp( item, args... ) App.i18n.translateTimestamp(item, args...)
# define linkify helper # define linkify helper
params.L = ( item ) -> params.L = (item) ->
if item && typeof item is 'string' if item && typeof item is 'string'
return App.Utils.linkify( item ) return App.Utils.linkify(item)
item item
# define config helper # define config helper
params.C = ( key ) -> params.C = (key) ->
App.Config.get( key ) App.Config.get(key)
# define session helper # define session helper
params.S = ( key ) -> params.S = (key) ->
App.Session.get( key ) App.Session.get(key)
# define address line helper # define address line helper
params.AddressLine = ( line ) -> params.AddressLine = (line) ->
return '' if !line return '' if !line
items = emailAddresses.parseAddressList(line) items = emailAddresses.parseAddressList(line)
@ -202,15 +200,14 @@ class App extends Spine.Controller
result = result + App.Utils.htmlEscape(item.name) + ' ' result = result + App.Utils.htmlEscape(item.name) + ' '
if item.address if item.address
result = result + " <span class=\"text-muted\">&lt;#{App.Utils.htmlEscape(item.address)}&gt</span>" result = result + " <span class=\"text-muted\">&lt;#{App.Utils.htmlEscape(item.address)}&gt</span>"
result result
# define file size helper # define file size helper
params.humanFileSize = ( size ) -> params.humanFileSize = (size) ->
App.Utils.humanFileSize(size) App.Utils.humanFileSize(size)
# define pretty/human time helper # define pretty/human time helper
params.humanTime = ( time, escalation = false, cssClass = '') -> params.humanTime = (time, escalation = false, cssClass = '') ->
timestamp = App.i18n.translateTimestamp(time) timestamp = App.i18n.translateTimestamp(time)
if escalation if escalation
cssClass += ' escalation' cssClass += ' escalation'
@ -222,7 +219,7 @@ class App extends Spine.Controller
App.Utils.icon(name, className) App.Utils.icon(name, className)
# define richtext helper # define richtext helper
params.RichText = ( string ) -> params.RichText = (string) ->
if string.match(/@T\('/) if string.match(/@T\('/)
string = string.replace(/@T\('(.+?)'\)/g, (match, capture) -> string = string.replace(/@T\('(.+?)'\)/g, (match, capture) ->
App.i18n.translateContent(capture) App.i18n.translateContent(capture)

View file

@ -27,6 +27,11 @@ class App.Log
_instance ?= new _Singleton _instance ?= new _Singleton
_instance.config( type, regex ) _instance.config( type, regex )
@timeTrack: (message) ->
if _instance == undefined
_instance ?= new _Singleton
_instance.timeTrack(message)
class _Singleton class _Singleton
constructor: -> constructor: ->
@moduleColorsMap = {} @moduleColorsMap = {}
@ -116,3 +121,13 @@ class _Singleton
@hue += goldenRatio @hue += goldenRatio
@hue = @hue % 1 @hue = @hue % 1
@hue * 360 @hue * 360
timeTrack: (message) =>
currentTime = new Date().getTime()
if !@lastTime
@lastTime = currentTime
console.log('timeTrack start', message)
else
diff = currentTime - @lastTime
@lastTime = currentTime
console.log('timeTrack start', message, diff)

View file

@ -41,7 +41,7 @@
<% groupLast = '' %> <% groupLast = '' %>
<% for object in @objects: %> <% for object in @objects: %>
<% if @groupBy: %> <% if @groupBy: %>
<% groupByName = @P( object, @groupBy ) %> <% groupByName = @P( object, @groupBy, @attributes ) %>
<% if groupLast isnt groupByName: %> <% if groupLast isnt groupByName: %>
<tr class=""><td colspan="<%= length %>"><b><%= groupByName %></b></td></tr> <tr class=""><td colspan="<%= length %>"><b><%= groupByName %></b></td></tr>
<% groupLast = groupByName %> <% groupLast = groupByName %>
@ -68,7 +68,7 @@
</td> </td>
<% end %> <% end %>
<% for item in @header: %> <% for item in @header: %>
<% value = @P( object, item.name ) %> <% value = @P( object, item.name, @attributes ) %>
<% if @callbacks: %> <% if @callbacks: %>
<% if item.name.substr(item.name.length-3, item.name.length) is '_id' && object[ item.name.substr(0, item.name.length-3) ]: %> <% if item.name.substr(item.name.length-3, item.name.length) is '_id' && object[ item.name.substr(0, item.name.length-3) ]: %>
<% refObject = object[ item.name.substr(0, item.name.length-3) ] %> <% refObject = object[ item.name.substr(0, item.name.length-3) ] %>