techjoomla / com_tjreports Goto Github PK

E.g.
Expected heading row

But right now it is getting created as

which disturbs the data in the subsequent rows

@monikaatvowel Can you check this?

In several cases, the report needs a complicated query to get the necessary data. In such cases the reports become highly non-performant.

Implement a way for plugins to save the "generated reports" in a separate table, so that the actual report can be shown based on this table. This will make the reports simple and performant.

cc @parthlawate @thite-amol @manojLondhe this has several similarities with the indexer. Do you see any way we can leverage indexer without brining all its baggage ?

After unchecking the column from the "Hide / Show Columns" drop-down option, the unchecked column gets removed from dropdown & does not display this option to show that column in a report.

The component was open to XSS attacks. Hence, this PR includes fixes for the same.

Need to add language const for labels in tjreport.xml file

Right now, though some columns are not shown in the backend reports, are still shown in the result when fetched with API plugin

TJ Reports fetches custom fields data twice in the report & it shows data twice the in TJ Reports API.

There are warnings in the reports model and reports default view.

Notice: Undefined index: record_id in /var/www/ttpllt-php72.local/public/jticketing/components/com_tjreports/models/reports.php on line 182

Notice: Undefined index: username_hash in /var/www/ttpllt-php72.local/public/jticketing/components/com_tjreports/models/reports.php on line 182

Notice: Undefined index: email_hash in /var/www/ttpllt-php72.local/public/jticketing/components/com_tjreports/models/reports.php on line 182

Warning: array_flip() expects parameter 1 to be array, null given in /var/www/ttpllt-php72.local/public/jticketing/components/com_tjreports/models/reports.php on line 547

Warning: array_keys() expects parameter 1 to be array, null given in /var/www/ttpllt-php72.local/public/jticketing/components/com_tjreports/views/reports/view.base.php on line 164

Please refer below image for the warnings.

Issue => As core component does not have techjoomla library dependency and still the javascript having techjoomla.jQuery stuff it gives javascript error.
How to reproduce = >

1. Install com_tjreport
2. Now create demo plugin for tjreport
3. Create menu at front end , select view as reports
4. Go to reports page and open console.log
5. You will see javascript error techjoomla not found.

Solution => As techjoomla dependency removed from core component we should not be using techjoomla.jQuery anywhere in components javascript.

Maintainance: move the plugins folder under the tjreports(src) folder

By default hide the custom field columns(record id + hashed columns) in reports but show them in "Hide / Show Columns" dropdown so that admin can set as per the requirement.