Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id 0A17A200C8B for ; Mon, 22 May 2017 10:25:17 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id 08A1C160BBF; Mon, 22 May 2017 08:25:17 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 4DF50160BA5 for ; Mon, 22 May 2017 10:25:16 +0200 (CEST) Received: (qmail 27070 invoked by uid 500); 22 May 2017 08:25:15 -0000 Mailing-List: contact notifications-help@ofbiz.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@ofbiz.apache.org Delivered-To: mailing list notifications@ofbiz.apache.org Received: (qmail 27061 invoked by uid 99); 22 May 2017 08:25:15 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd1-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 22 May 2017 08:25:15 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd1-us-west.apache.org (ASF Mail Server at spamd1-us-west.apache.org) with ESMTP id 09BB6C660E for ; Mon, 22 May 2017 08:25:15 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd1-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: -98.689 X-Spam-Level: X-Spam-Status: No, score=-98.689 tagged_above=-999 required=6.31 tests=[RP_MATCHES_RCVD=-0.001, SPF_PASS=-0.001, URI_HEX=1.313, USER_IN_WHITELIST=-100] autolearn=disabled Received: from mx1-lw-eu.apache.org ([10.40.0.8]) by localhost (spamd1-us-west.apache.org [10.40.0.7]) (amavisd-new, port 10024) with ESMTP id CgkORL_2k_IU for ; Mon, 22 May 2017 08:25:12 +0000 (UTC) Received: from mailrelay1-us-west.apache.org (mailrelay1-us-west.apache.org [209.188.14.139]) by mx1-lw-eu.apache.org (ASF Mail Server at mx1-lw-eu.apache.org) with ESMTP id 455D85FC57 for ; Mon, 22 May 2017 08:25:07 +0000 (UTC) Received: from jira-lw-us.apache.org (unknown [207.244.88.139]) by mailrelay1-us-west.apache.org (ASF Mail Server at mailrelay1-us-west.apache.org) with ESMTP id 4DADCE0BCA for ; Mon, 22 May 2017 08:25:06 +0000 (UTC) Received: from jira-lw-us.apache.org (localhost [127.0.0.1]) by jira-lw-us.apache.org (ASF Mail Server at jira-lw-us.apache.org) with ESMTP id 0AB8E21B5A for ; Mon, 22 May 2017 08:25:05 +0000 (UTC) Date: Mon, 22 May 2017 08:25:05 +0000 (UTC) From: "Jacques Le Roux (JIRA)" To: notifications@ofbiz.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Issue Comment Deleted] (OFBIZ-9351) Remove unnecessary field types MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 archived-at: Mon, 22 May 2017 08:25:17 -0000 [ https://issues.apache.org/jira/browse/OFBIZ-9351?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jacques Le Roux updated OFBIZ-9351: ----------------------------------- Comment: was deleted (was: Replaced fields with "id-ne", "id-long-ne" and "id-vlong-ne" with "id", "id-long" and "id-vlong" respectively which are primary keys. The patch also covers the field names like "parentTypeId" as it will always be empty for some record that starts the hierarchy.) > Remove unnecessary field types > ------------------------------ > > Key: OFBIZ-9351 > URL: https://issues.apache.org/jira/browse/OFBIZ-9351 > Project: OFBiz > Issue Type: Task > Reporter: Aditya Sharma > Assignee: Aditya Sharma > Priority: Trivial > Attachments: OFBIZ-9351.patch, OFBIZ-9351_plugins.patch > > > As discussed in http://markmail.org/message/d2tpovewhtotukwa the "not empty" field types ("id-ne", "id-long-ne" and "id-vlong-ne") were initially added to implement validations on data. But, because the validations where only implemented in some place like webtools, it contradicts the distinction to be upheld on various layers. So it is better to remove these field types. So we will remove all those field types and will create sub tickets to replace them in entity definitions by corresponding "id" types. > When replacing the "not empty" field types in entity definitions by corresponding "id" types we will add a *not-null="true"* attribute to "makes the field NOT NULL on the database (like primary key fields)" as explained in "not-null" documentation in fieldtypemodel.xsd. We will finally clean the documentation of the "not-null" in fieldtypemodel.xsd. > Related Links: > http://ofbiz.135035.n4.nabble.com/EntityEngine-field-types-td2251546.html > http://markmail.org/message/otec62xiwkpjttkq > A more vivid description: > http://ofbiz.markmail.org/thread/c6ee3ewyo6jpik7k -- This message was sent by Atlassian JIRA (v6.3.15#6346)