Uploaded image for project: 'OSAMES ORM'
  1. OSAMES ORM
  2. ORM-169

Envoyer des valeurs NULL vers la base de données au lieu de chaîne vide

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.1.0
    • Fix Version/s: 0.2.0
    • Component/s: None
    • Labels:
      None

      Description

      Lorsqu'une propriété d'un IDatabaseEntityObject a pour valeur chaîne vide, alors la valeur du paramètre ADO.NET correspondant doit être NULL et non chaîne vide.

      Cela permet de mieux valider : champ vide sur le formulaire = pas de valeur = erreur si champ obligatoire en DB. Au lieu d'enregistrer une chaîne vide qui passe dans ce cas.

      Method DetermineDatabaseColumnNamesAndAdoParameters ligne 124 (et pareil ligne 87) :
      Unable to find source-code formatter for language: csharp. Available languages are: actionscript, ada, applescript, bash, c, c#, c++, cpp, css, erlang, go, groovy, haskell, html, java, javascript, js, json, lua, none, nyan, objc, perl, php, python, r, rainbow, ruby, scala, sh, sql, swift, visualbasic, xml, yaml
      lstAdoParameterNameAndValues_.Add(new KeyValuePair<string, object>("@" + propertyName.ToLowerInvariant(), databaseEntityObjectProperty.GetValue(databaseEntityObject_)));
      

      File DbToolsCommon
      Concerns Update et Insert

      Remarque : il existe un test unitaire TestUpdateSingleErrorWithoutMandatoryValuesSqlite dans TestDBToolsUpdates qui concerne l'enregistrement d'une valeur obligatoire en DB qu'on passe à NULL mais ce n'est pas le même contexte qu'ici, il faudra donc ajouter des tests (en premier lieu sur le formatage fait par DbToolsCommon où on s'attendra à recevoir un objet NULL dans le tableau de valeurs, ou un paramètre ADO.NET avec comme valeur System.DbNull).

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                babs Barbara Post
                Reporter:
                babs Barbara Post
              • Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: