• last updated 5 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
New test checking behavior of the news aggregator

This currently fails for what concerns updating an existing news item

    • -0
    • +90
    ./tcl/test/news-aggregator-test-procs.tcl
Use existing api

Prepend source URL to the one of the news item in case this is not an absolute URL

Add test for the news aggregator and feed-parser

news-aggregator will reuse basic test from the feed parser

    • -0
    • +46
    ./tcl/test/news-aggregator-test-procs.tcl
  1. … 1 more file in changeset.
Fix subscription form error (left behind from previous commit)

Trim whitespace

Repair form validation of URL

New test to check whether configured news feeds look ok

Trim whitespace

Replace deprecated header_stuff with head

Give some feedback in case of error

Repair broken opml export, exploit new api to streamline query, fix typo

    • -17
    • +0
    ./tcl/aggregator-procs-postgresql.xql
Return 0 also when a default is there but is the empty string

Bring other sql inline, simplify idioms

    • -49
    • +0
    ./tcl/aggregator-procs-postgresql.xql
Bring sql for this proc inline, remove unused statements

    • -41
    • +0
    ./tcl/aggregator-procs-postgresql.xql
Document public procs (fixes automated tests), rework code, remove leftover statements

Modernize tcl

Trim whitespace

Remove obsolete xql files

    • -34
    • +0
    ./tcl/subscription-procs-postgresql.xql
Make all sql portable and bring it in line, remove code duplication

Further UI fixing after tests to make so that actions are displayed only when the proper conditions are met

Revision of the index page:

- pass template properties according to conventions

- streamline boolean expressions

- fix UI, show purge button and its explanation when the actual form is generated

- make sql portable and bring it inline

- better exploit api

Use db_0or1row flag to return an array natively, repair news_aggregator::aggregator::as_xml as for some reason creating the xml processing instruction via tDOM is broken on tDOM 0.8.3

    • -10
    • +0
    ./tcl/aggregator-procs-postgresql.xql
Whitespace cleanup

Fix boolean adp expression, pass properties according to adp guidelines

Package rework and maintenance

- deprecate undocumented news_aggregator::source::get_identifier as no mention was found in upstream code (fixes automated tests)

- put all portable sql inline

- rework all sql idioms that could be made portable

- remove leftover sql statements

- simplify idioms

Rework of news_aggregator::source::update

- replace deprecated calls

- replace array idioms with dict idioms

- bring portable sql inline

This was tested against RSS feeds used downstream on a vanilla instance

Change adp expression so news can be displayed

Removed 'setResultEncoding' tdom option, which has no effect in Tcl > 8.0, and is removed in tdom > 0.9.0.

See upstream commit: http://tdom.org/index.html/info/d729de4c396d71ff

From the old docs: http://tdom.org/index.html/artifact/2a79d8cf33feacf9

dom setResultEncoding ?encodingName?

This option is for backward compatibility with Tcl 8.0. If tDOM is build

with any newer Tcl version this option does not has any effect.

If encodingName is not given the current global result encoding is

returned. Otherwise the global result encoding is set to encodingName.

All character data, attribute values etc. will then be converted from

UTF-8, which is delivered from the Expat XML parser, to the given 8 bit

encoding at XML/DOM parse time. Valid values for encodingName are:

utf-8, ascii, cp1250, cp1251, cp1252, cp1253, cp1254, cp1255, cp1256,

cp437, cp850, en, iso8859-1, iso8859-2, iso8859-3, iso8859-4, iso8859-5,

iso8859-6, iso8859-7, iso8859-8, iso8859-9, koi8-r.

Almost complete rewrite of news_aggregator::source::new

- replace deprecated calls

- remove code duplication

- don't loop through response items, as nothing is done with that information

- replace array idioms with dict idioms

- let stored procedure figure out the object_id

Works on WU RSS URLs on a vanilla instance