Comments (6)
Text is now saved. Problem with field of type picture.
from moodle-mod_datalynx.
TODO: Do not save empty fields in contents table (example: empty string). Delete those rows in DB instead or do not consider those entries as "existing entries" or data.
from moodle-mod_datalynx.
The assumption things have been "visible to user" is not a valid assumption and is one of the origins of this bug. The concept to ignore certain submitted data is very dangerous. All submitted data has to be analysed.
https://github.com/dasistwas/moodle-mod_datalynx/blob/0fddcff6bbcb6cba7ccd94adf5066cf8daea1300/entries_class.php#L753
The iterator does not work either as the foreach is not in line with $i ($i differs from actual line number in the nested foreach loop).
from moodle-mod_datalynx.
Better approach could be: delete content from db when no data is submitted for the field.
from moodle-mod_datalynx.
What does not work: Removing the teammembers with the JavaScript "Delete" button does not work. They still appear when saving the form.
Next step: Strategy for displaying and saving data in a solid way. Open for discussion:
Approach 1
- Add integer field to datalynx_contents table named rowid or lineid
- Store the line of the fieldgroup there
- Make deletion of content therefore easier: delete datalanx_contents where fieldid, entryid, rowid
from moodle-mod_datalynx.
Rebuilt this as discussed.
We now process all data, collect where we do see empty fieldvalues and check if a line is completely empty.
If this is the case we remove all line contents from database.
I'm closing this to allow for more precise issue descriptions, filed issue is fixed.
from moodle-mod_datalynx.
Related Issues (20)
- Teammembers can edit entries HOT 1
- Hide uploaded files from other users. HOT 4
- Pattern replace within attributes does not work anymore HOT 17
- Adding data-* attribute in renderer does not work HOT 4
- Error on new entry HOT 3
- Notice on export/import view HOT 1
- Duplicate rule results in error
- Merge develop branch HOT 1
- CSV export of the course already posted here throws those notices HOT 1
- Recently upgraded datalynx instance throws an error when I want to edit a text field HOT 2
- Add top level course category id to body class in view.php
- Deleted users are still teammembers in temamemberselect field
- When opening statistics tab in datalynx the error is shown when no end date is set
- Possible Error with file storage on 3.11 HOT 2
- Read from DB error when clicking two times on Convert column in Datalynx instance -> Manage -> Fields
- Grid settings to use 1 column HOT 2
- Mandatory [[*required]] fields with tags not working? HOT 3
- Time field option 'display format' does not render if 'Date only' is selected HOT 2
- CSV View export did not adhere to filters applied
- Creating CSV view fails with implode error HOT 7
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from moodle-mod_datalynx.